-
Notifications
You must be signed in to change notification settings - Fork 30
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
Compatibility with the python version #2
Comments
It’s not 100% compatible with the Python version but it’s close to that. |
I asked because it has the same name as the python version and I think it should be noted somewhere if it's compatible and if it will stay this way. If it's compatible it can be used to port applications from python to rust, if they are using PickleDb. If compatibility is broken, it would probably be better to change the name to something different, so that there will be no confusion between the two. |
Currently it's almost compatible. Of course Rust and Python are different programming languages so it'll never actually be 100% compatibility. I agree with you that if I decide to change the interface it'd be a good idea to change the name too |
Please let me know if we can close this issue |
I'm closing this issue for now. Please reopen it if needed |
Standard process for an atomic update on unix systems is: 1. Write new contents to a temp file on same filesystem/folder 2. fsync the file 3. Close the file 4. Rename the temp file over top of the existing file The existing code was missing seladb#2. If you lose power soon after persisting a new file, the file rename can be synced through without the new data contents. If that happens you end up losing your settings.
Is this 100% compatible with the python version? And if so, are there plans to keep it compatible?
The text was updated successfully, but these errors were encountered: