search

ex0dus-0x / microkv

Minimal and persistent key-value store designed with security in mind
MIT License
32 stars 6 forks source link

support namespace #10

Closed fewensa closed 2 years ago

fewensa commented 3 years ago

The namespace can isolate storage.

In addition, a dependency dirs is added, because std::env::var("HOME") does not support windows

fewensa commented 3 years ago

Maybe it is more appropriate to use different storage for different namespaces?

fewensa commented 3 years ago

I changed namespace use different storage. and supports data migration between different versions https://github.com/fewensa/microkv/pull/1 Hope to submit pr to you after seeing your comments

ex0dus-0x commented 2 years ago

Huge apologies for the long wait on merging this, as other work was prioritized!

Never really thought about the need for namespacing/buckets for the purposes this project was introduced for, but seeing the demand in other projects, will definitely merge to fully support.

Little iffy about introducing another dirs dependency, but will introduce for cross-platform compatibility.

fewensa commented 2 years ago

In fact, I made a lot of changes after this pr, including the migration after the version upgrade, etc., which became more complicated than before. But I think it can be optimized a bit, and I will submit the pr to you after the modification is completed. If you think it’s okay, I hope it can be merged