1. 23 11月, 2017 1 次提交
  2. 22 11月, 2017 2 次提交
  3. 21 11月, 2017 5 次提交
  4. 20 11月, 2017 1 次提交
  5. 18 11月, 2017 2 次提交
  6. 17 11月, 2017 1 次提交
  7. 16 11月, 2017 5 次提交
  8. 15 11月, 2017 1 次提交
    • M
      Properly destruct rebuilding_trx_ · 175d5d6a
      Maysam Yabandeh 提交于
      Summary:
      When testing rebuilding_trx_ in MemTableInserter might still be set before the tests finishes which would cause ASAN alarms for leaks. This patch deletes the pointers in MemTableInserter destructor.
      Closes https://github.com/facebook/rocksdb/pull/3162
      
      Differential Revision: D6317113
      
      Pulled By: maysamyabandeh
      
      fbshipit-source-id: a68be70709a4fff7ac2b768660119311968f9c21
      175d5d6a
  9. 14 11月, 2017 2 次提交
  10. 12 11月, 2017 2 次提交
  11. 11 11月, 2017 4 次提交
    • M
      WritePrepared Txn: Refactor conf params · 857adf38
      Maysam Yabandeh 提交于
      Summary:
      Summary of changes:
      - Move seq_per_batch out of Options
      - Rename concurrent_prepare to two_write_queues
      - Add allocate_seq_only_for_data_
      Closes https://github.com/facebook/rocksdb/pull/3136
      
      Differential Revision: D6304458
      
      Pulled By: maysamyabandeh
      
      fbshipit-source-id: 08e685bfa82bbc41b5b1c5eb7040a8ca6e05e58c
      857adf38
    • A
      prefer enabling cpu features via -march/-mcpu · 07c2738f
      Andrew Kryczka 提交于
      Summary:
      If possible, use -march or -mcpu to get enable all features available on the local CPU or architecture. Only if this is impossible, we will manually set -msse4.2. It should be safe as there'll be a warning printed if `USE_SSE` is set and the provided flags are insufficient to support SSE4.2.
      Closes https://github.com/facebook/rocksdb/pull/3156
      
      Differential Revision: D6304703
      
      Pulled By: ajkr
      
      fbshipit-source-id: 030a53491263300cae7fafb429114d87acc828ef
      07c2738f
    • D
      Fix crashes, address test issues and adjust windows test script · f8e2db07
      Dmitri Smirnov 提交于
      Summary:
      Add per-exe execution capability
        Add fix parsing of groups/tests
        Add timer test exclusion
      
       Fix unit tests
        Ifdef threadpool specific tests that do not pass on Vista threadpool.
        Remove spurious outout from prefix_test so test case listing works
        properly.
        Fix not using standard test directories results in file creation errors
        in sst_dump_test.
      
        BlobDb fixes:
          In C++ end() iterators can not be dereferenced. They are not valid.
      	When deleting blob_db_ set it to nullptr before any other code executes.
      	Not fixed:. On Windows you can not delete a file while it is open.
      	[ RUN      ] BlobDBTest.ReadWhileGC
      	d:\dev\rocksdb\rocksdb\utilities\blob_db\blob_db_test.cc(75): error: DestroyBlobDB(dbname_, options, bdb_options)
      	IO error: Failed to delete: d:/mnt/db\testrocksdb-17444/blob_db_test/blob_dir/000001.blob: Permission denied
      	d:\dev\rocksdb\rocksdb\utilities\blob_db\blob_db_test.cc(75): error: DestroyBlobDB(dbname_, options, bdb_options)
      	IO error: Failed to delete: d:/mnt/db\testrocksdb-17444/blob_db_test/blob_dir/000001.blob: Permission denied
      
        write_batch
          Should not call front() if there is a chance the container is empty
      Closes https://github.com/facebook/rocksdb/pull/3152
      
      Differential Revision: D6293274
      
      Pulled By: sagar0
      
      fbshipit-source-id: 318c3717c22087fae13b18715dffb24565dbd956
      f8e2db07
    • S
      Stream · eefd75a2
      Shaohua Li 提交于
      Summary:
      Add a simple policy for NVMe write time life hint
      Closes https://github.com/facebook/rocksdb/pull/3095
      
      Differential Revision: D6298030
      
      Pulled By: shligit
      
      fbshipit-source-id: 9a72a42e32e92193af11599eb71f0cf77448e24d
      eefd75a2
  12. 10 11月, 2017 2 次提交
  13. 09 11月, 2017 2 次提交
    • Y
      Blob DB: Fix race condition between flush and write · 5e9e5a47
      Yi Wu 提交于
      Summary:
      A race condition will happen when:
      * a user thread writes a value, but it hits the write stop condition because there are too many un-flushed memtables, while holding blob_db_impl.write_mutex_.
      * Flush is triggered and call flush begin listener and try to acquire blob_db_impl.write_mutex_.
      
      Fixing it.
      Closes https://github.com/facebook/rocksdb/pull/3149
      
      Differential Revision: D6279805
      
      Pulled By: yiwu-arbug
      
      fbshipit-source-id: 0e3c58afb78795ebe3360a2c69e05651e3908c40
      5e9e5a47
    • Y
      Blob DB: Fix release build · ca75f0a6
      Yi Wu 提交于
      Summary:
      `compression` shadow the method name in `BlobFile`. Rename it.
      Closes https://github.com/facebook/rocksdb/pull/3148
      
      Differential Revision: D6274498
      
      Pulled By: yiwu-arbug
      
      fbshipit-source-id: 7d293596530998b23b6b8a8940f983f9b6343a98
      ca75f0a6
  14. 08 11月, 2017 3 次提交
  15. 07 11月, 2017 1 次提交
    • M
      Add lock wait time as a perf context counter · e03377c7
      Manuel Ung 提交于
      Summary:
      Adds two new counters:
      
      `key_lock_wait_count` counts how many times a lock was blocked by another transaction and had to wait, instead of being granted the lock immediately.
      `key_lock_wait_time` counts the time spent acquiring locks.
      Closes https://github.com/facebook/rocksdb/pull/3107
      
      Differential Revision: D6217332
      
      Pulled By: lth
      
      fbshipit-source-id: 55d4f46da5550c333e523263422fd61d6a46deb9
      e03377c7
  16. 04 11月, 2017 6 次提交
    • Y
      Fix PinnableSlice move assignment · be410ded
      Yi Wu 提交于
      Summary:
      After move assignment, we need to re-initialized the moved PinnableSlice.
      
      Also update blob_db_impl.cc to not reuse the moved PinnableSlice since it is supposed to be in an undefined state after move.
      Closes https://github.com/facebook/rocksdb/pull/3127
      
      Differential Revision: D6238585
      
      Pulled By: yiwu-arbug
      
      fbshipit-source-id: bd99f2e37406c4f7de160c7dee6a2e8126bc224e
      be410ded
    • S
      Remove unnecessary status check in TableCache::NewIterator · a6d8e30c
      Sagar Vemuri 提交于
      Summary:
      While investigating the usage of `new_table_iterator_nanos` perf counter, I saw some code was wrapper around with unnecessary status check ... so removed it.
      Closes https://github.com/facebook/rocksdb/pull/3120
      
      Differential Revision: D6229181
      
      Pulled By: sagar0
      
      fbshipit-source-id: f8a44fe67f5a05df94553fdb233b21e54e88cc34
      a6d8e30c
    • P
      util: Fix coverity issues · 4c8f3364
      Prashant D 提交于
      Summary:
      util/concurrent_arena.h:
      CID 1396145 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
      2. uninit_member: Non-static class member free_begin_ is not initialized in this constructor nor in any functions that it calls.
       94    Shard() : allocated_and_unused_(0) {}
      
      util/dynamic_bloom.cc:
      	1. Condition hash_func == NULL, taking true branch.
      
      CID 1322821 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
      3. uninit_member: Non-static class member data_ is not initialized in this constructor nor in any functions that it calls.
      47      hash_func_(hash_func == nullptr ? &BloomHash : hash_func) {}
      48
      
      util/file_reader_writer.h:
      204 private:
      205  AlignedBuffer buffer_;
         	member_not_init_in_gen_ctor: The compiler-generated constructor for this class does not initialize buffer_offset_.
      206  uint64_t buffer_offset_;
      
      CID 1418246 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
      member_not_init_in_gen_ctor: The compiler-generated constructor for this class does not initialize buffer_len_.
      207  size_t buffer_len_;
      208};
      
      util/thread_local.cc:
      341#endif
      
      CID 1322795 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
      3. uninit_member: Non-static class member pthread_key_ is not initialized in this constructor nor in any functions that it calls.
      342}
      
      40struct ThreadData {
         	2. uninit_member: Non-static class member next is not initialized in this constructor nor in any functions that it calls.
      
      CID 1400668 (#1 of 1): Uninitialized pointer field (UNINIT_CTOR)
      4. uninit_member: Non-static class member prev is not initialized in this constructor nor in any functions that it calls.
       41  explicit ThreadData(ThreadLocalPtr::StaticMeta* _inst) : entries(), inst(_inst) {}
       42  std::vector<Entry> entries;
         	1. member_decl: Class member declaration for next.
       43  ThreadData* next;
         	3. member_decl: Class member declaration for prev.
       44  ThreadData* prev;
       45  ThreadLocalPtr::StaticMeta* inst;
       46};
      Closes https://github.com/facebook/rocksdb/pull/3123
      
      Differential Revision: D6233566
      
      Pulled By: sagar0
      
      fbshipit-source-id: aa2068790ea69787a0035c0db39d59b0c25108db
      4c8f3364
    • A
      fix CopyFile status checks · cfb120f7
      Andrew Kryczka 提交于
      Summary:
      copied from internal diff D6156261
      Closes https://github.com/facebook/rocksdb/pull/3124
      
      Differential Revision: D6230167
      
      Pulled By: ajkr
      
      fbshipit-source-id: 17926bb1152d607556364e3aacfec0ef3c115748
      cfb120f7
    • Y
      Fix clang build error · d9561695
      Yi Wu 提交于
      Summary:
      Fix cast from size_t to unsigned int.
      Closes https://github.com/facebook/rocksdb/pull/3125
      
      Differential Revision: D6232863
      
      Pulled By: yiwu-arbug
      
      fbshipit-source-id: 4c6131168b1faec26f7820b2cf4a09c242d323b7
      d9561695
    • Y
      Blob DB: Fix BlobDBTest::SnapshotAndGarbageCollection asan failure · 2581c0a5
      Yi Wu 提交于
      Summary:
      Fix unreleased snapshot at the end of the test.
      Closes https://github.com/facebook/rocksdb/pull/3126
      
      Differential Revision: D6232867
      
      Pulled By: yiwu-arbug
      
      fbshipit-source-id: 651ca3144fc573ea2ab0ab20f0a752fb4a101d26
      2581c0a5