Vincent Weevers
Vincent Weevers
I prefer for this to be fixed [upstream](https://github.com/google/leveldb), so that we don't have to maintain patches here. If it's already fixed upstream (not unlikely, because we use an old version...
Same issue as https://github.com/Level/rocksdb/issues/185#issuecomment-964857602, I believe. A pull request is welcome
Looks like this was a bug in node, fixed in https://github.com/nodejs/node/pull/47860.
> or should I use thgrough2, which is probably not using readable-stream under the hood `through2` has no reason to exist anymore. It does use `readable-stream` but it's outdated and...
> I do wondet what your // .. represents. I just intended to show that the body of the function is equal between the two code snippets. Fully written out,...
Superseded by https://github.com/Level/rave-level, didn't update the README yet (PR welcome).
We have an upgrade guide here (for replacing `level-party` with `rave-level`): https://github.com/Level/rave-level/blob/main/UPGRADING.md
Optimized it by using an iterator instead of many `Get()` calls, and a bitset instead of many `napi_set_element` calls.
Does it merely create the directory or are there also LevelDB files inside? Which platform? We have unit tests for this with one known issue: https://github.com/Level/classic-level/blob/15eb2893086ff52f6ce5b2b79a21ca6340b966fc/test/mkdir-test.js#L32-L33
> in classic-level/index.js it is forced to true If you're referring to the following line - that's not the option. This is a so-called manifest that tells `abstract-level` that this...