cmake: append rados to THIRDPARTY_LIBS before appending it to LIBS
Summary:
otherwise the env_librados_test executable will fail to link against
librados.
Signed-off-by: Kefu Chai <tchaikov@gmail.com>
Closes https://github.com/facebook/rocksdb/pull/3724
Differential Revision:
D7631542
Pulled By: ajkr
fbshipit-source-id:
38afbf21f9aeb7dedfb840aba8b2f8b421f9edb0
Fix up backupable_db stack corruption.
Summary:
Fix up OACR(Lint) warnings.
Closes https://github.com/facebook/rocksdb/pull/3674
Differential Revision:
D7563869
Pulled By: ajkr
fbshipit-source-id:
8c1e5045c8a6a2d85b2933fdbc60fde93bf0c9de
Fix the memory leak with pinned partitioned filters
Summary:
The existing unit test did not set the level so the check for pinned partitioned filter/index being properly released from the block cache was not properly exercised as they only take effect in level 0. As a result a memory leak in pinned partitioned filters was hidden. The patch fix the test as well as the bug.
Closes https://github.com/facebook/rocksdb/pull/3692
Differential Revision:
D7559763
Pulled By: maysamyabandeh
fbshipit-source-id:
55eff274945838af983c764a7d71e8daff092e4a
Change a comment
Summary:
In this case, we add input files of compaction, not outputs.
Closes https://github.com/facebook/rocksdb/pull/3686
Differential Revision:
D7556781
Pulled By: ajkr
fbshipit-source-id:
ae135bb6eda60db8f275a9ba2d21c18aaadef5b7
fix intra-L0 FIFO for uncompressed use case
Summary:
- inflate the argument passed as `max_compact_bytes_per_del_file` by a bit (10%). The intent of this argument is prevent L0 files from being intra-L0 compacted multiple times. Without compression, some intra-L0 compactions exceed this limit (and thus aren't executed), even though none of their files have gone through intra-L0 before.
- fix `FindIntraL0Compaction` as it was rejecting some valid intra-L0 compactions. In particular, `compact_bytes_per_del_file` is the work-per-deleted-file for the span [0, span_len), whereas `new_compact_bytes_per_del_file` is the work-per-deleted-file for the span [0, span_len+1). The former is more correct for checking whether we've found an eligible span.
Closes https://github.com/facebook/rocksdb/pull/3684
Differential Revision:
D7530396
Pulled By: ajkr
fbshipit-source-id:
cad4f50902bdc428ac9ff6fffb13eb288648d85e
fix data race
Summary:
Fix a TSAN failure in `DBRangeDelTest.ValidLevelSubcompactionBoundaries`:
https://gist.github.com/miasantreble/
712e04b4de2ff7f193c98b1acf07e899
Closes https://github.com/facebook/rocksdb/pull/3691
Differential Revision:
D7541400
Pulled By: miasantreble
fbshipit-source-id:
b0b4538980bce7febd0385e61d6e046580bcaefb
WritePrepared Txn: add stats
Summary:
Adding some stats that would be helpful to monitor if the DB has gone to unlikely stats that would hurt the performance. These are mostly when we end up needing to acquire a mutex.
Closes https://github.com/facebook/rocksdb/pull/3683
Differential Revision:
D7529393
Pulled By: maysamyabandeh
fbshipit-source-id:
f7d36279a8f39bd84d8ddbf64b5c97f670c5d6d9
WritePrepared Txn: add write_committed option to dump_wal
Summary:
Currently dump_wal cannot print the prepared records from the WAL that is generated by WRITE_PREPARED write policy since the default reaction of the handler is to return NotSupported if markers of WRITE_PREPARED are encountered. This patch enables the admin to pass --write_committed=false option, which will be accordingly passed to the handler. Note that DBFileDumperCommand and DBDumperCommand are still not updated by this patch but firstly they are not urgent and secondly we need to revise this approach later when we also add WRITE_UNPREPARED markers so I leave it for future work.
Tested by running it on a WAL generated by WRITE_PREPARED:
$ ./ldb dump_wal --walfile=/dev/shm/dbbench/000003.log | grep BEGIN_PREARE | head -1
1,2,70,0,BEGIN_PREARE
$ ./ldb dump_wal --walfile=/dev/shm/dbbench/000003.log --write_committed=false | grep BEGIN_PREARE | head -1
1,2,70,0,BEGIN_PREARE PUT(0) : 0x30303031313330313938 PUT(0) : 0x30303032353732313935 END_PREPARE(0x74786E31313535383434323738303738363938313335312D30)
Closes https://github.com/facebook/rocksdb/pull/3682
Differential Revision:
D7522090
Pulled By: maysamyabandeh
fbshipit-source-id:
a0332207261c61e18b2f9dfbe9feecd9a1339aca
Added support for SstFileManager to RocksJava
Summary: Closes https://github.com/facebook/rocksdb/pull/3666
Differential Revision:
D7457634
Pulled By: sagar0
fbshipit-source-id:
47741e2ee66e9255c580f4e38cfb86b284c27c2f
Fix typo
Summary:
regrad -> regard
Closes https://github.com/facebook/rocksdb/pull/3685
Differential Revision:
D7540952
Pulled By: miasantreble
fbshipit-source-id:
e08c9389f7fccf401c962a4441b62cd5e73a33ad