-
Notifications
You must be signed in to change notification settings - Fork 21
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
jammdb rewrite #17
jammdb rewrite #17
Conversation
3cc27f7
to
c728f73
Compare
Codecov ReportBase: 92.90% // Head: 94.74% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #17 +/- ##
==========================================
+ Coverage 92.90% 94.74% +1.83%
==========================================
Files 18 21 +3
Lines 1790 3614 +1824
==========================================
+ Hits 1663 3424 +1761
- Misses 127 190 +63
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
Complete rewrite of the database to remove a ton of unsafe code.
Deleted the Ptr struct, and store buckets and nodes in a more idiomatic way rather than using unsafe pointers.
Stops copying the bytes written to the database, to vectors and using unsafe pointers to refer to them. Instead uses the ToBytes trait to safely accept a &[u8] as long as it outlives the transaction. You can also pass in a Vec or a bytes::Bytes and they will not be copied.