Andrew Kryczka

> Zig-zag encoded values are in turn encoded as varints in the data. Varints now implement the 8-bit variable-length encoding (known length) Why do we need to zig-zag? Now that...

Not sure. Empty full_history_ts_low seems to unambiguously mean none is set. There might need to be a reason to change it. Then if we do change it, we'll need a...

I think that wiki is outdated. That pseudocode is nine years old. I believe we intend to call `FullMergeV2()` when the operand list is known to include the oldest possible...

Your finding is expected and correct. Only full merge should be called during `Get()`. The merge needs to be full to return a result; adding partial merge during `Get()` is...

Consider picking from our up-for-grabs issues: https://github.com/facebook/rocksdb/issues?q=is%3Aissue+is%3Aopen+label%3Aup-for-grabs

@yizhang82 Would it be easier if I submit these as Phabricator diffs?

Thanks, it turns out this is a known problem. Will use #5763 for tracking this issue.

> @ajkr When can this be merged? Thank you It needs a second employee to accept the internal diff. I will try to make sure that happens in time for...

I tried `./ldb compact --db=./tmp-db/ --column_family=column_family_name_000009`, and verified it compacted "column_family_name_000009" and nothing else

I see the appeal of exporting all the functions as librocksdbjni is *almost* usable as a general-purpose (non-Java) RocksDB library. But I am not sure we want to encourage or...