Rust-LSM-KV icon indicating copy to clipboard operation
Rust-LSM-KV copied to clipboard
verified publisher icon brucechin

Final code feedback

Open tov opened this issue 5 years ago • 0 comments

Looks like you’ve done an impressive job here. There are two things limiting this to a ✓: 1) Your testing appears perfunctory—this is not what testing well looks like. 2) Most of the methods are undocumented! Nice job overall, though :)

Hope you both stay healthy and enjoy your spring breaks.

J

tov avatar Mar 23 '20 07:03 tov