search

dermesser / leveldb-rs

A reimplementation of LevelDB in Rust (no bindings).
Other
521 stars 59 forks source link

skipmap: implement drop for the skip map to avoid stackoverflow caused by drop #21

Closed cfzjywxk closed 1 year ago

cfzjywxk commented 1 year ago

Close #20

Try to implement drop for the skip map to avoid stack overflow caused by drop, and cargo fmt some code.

Signed-off-by: cfzjywxk lsswxrxr@163.com

dermesser commented 1 year ago

otherwise fine, thank you so much :D

dermesser commented 1 year ago

Thank you, this is a great improvement! I myself hadn't seen this kind of failure before, but I guess my databases just haven't been large enough.