-
Notifications
You must be signed in to change notification settings - Fork 505
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor Tsavorite checkpoint path handling to be OS-aware #306
Refactor Tsavorite checkpoint path handling to be OS-aware #306
Conversation
* Use Path.Join() for OS-aware paths.
Something odd happening with the paths used in the cloud tests. Probably related to the Path.Join behavior which preserves seperators. |
* Let's see if this was the issue
* Just testing around, this might be the fix though
The problem is the fact that Tsavorite previously assumed all path-seperators to be This |
* Add AzureCheckpointNamingScheme which uses / as the checkpoint path seperator
Making rest of the Tsavorite tests use |
Actually, I'll have that in separate PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
No description provided.