1. 26 3月, 2015 3 次提交
    • J
      TSAN: avoid new link failure with -pg · ff1ff7c6
      Jim Meyering 提交于
      Summary:
      * Makefile (COMPILE_WITH_TSAN): Avoid a link failure by disabling
      -pg when building with TSAN enabled.
      Now that "make check" builds all $(PROGRAMS), it is linking
      a few programs that were not normally linked before.
      For example, this would fail to link with the following diagnostic:
      
        COMPILE_WITH_TSAN=1 make -j40 log_and_apply_bench
      
          CCLD     log_and_apply_bench
        ld: /usr/lib/../lib64/gcrt1.o: relocation R_X86_64_32S against `__libc_csu_fini' can not be used when making a shared object; recompile with -fPIC
        /usr/lib/../lib64/gcrt1.o: error adding symbols: Bad value
        collect2: error: ld returned 1 exit status
        Makefile:511: recipe for target 'log_and_apply_bench' failed
        make: *** [log_and_apply_bench] Error 1
      
      Since removing -pg is sufficient to get past this link
      failure, and no one cares about profiling TSAN-enabled
      binaries anyway, we will refrain from linking with -pg
      when TSAN testing is enabled.  Use a new variable, "pg"
      which is set to "-pg" in most cases, but that is made
      empty when COMPILE_WITH_TSAN is set.
      
      Test Plan:
        Now, this succeeds:
      
          rm -f log_and_apply_bench
          COMPILE_WITH_TSAN=1 make -j40 log_and_apply_bench
      
      Reviewers: igor.sugak, rven, sdong, ljin, igor
      
      Reviewed By: igor
      
      Subscribers: dhruba
      
      Differential Revision: https://reviews.facebook.net/D35943
      ff1ff7c6
    • Y
      Add a missing section title in HISTORY.md · 39d508e3
      Yueh-Hsuan Chiang 提交于
      Summary:
      Add a missing section title in HISTORY.md
      
      Test Plan:
      no code change
      39d508e3
    • Y
      Fixed a typo in RocksDBSample.java · 727684bf
      Yueh-Hsuan Chiang 提交于
      Summary:
      Fixed a typo in RocksDBSample.java
      
      Test Plan:
      make clean
      make rocksdbjava -j32
      make jtest
      727684bf
  2. 25 3月, 2015 16 次提交
  3. 24 3月, 2015 10 次提交
  4. 23 3月, 2015 2 次提交
  5. 21 3月, 2015 1 次提交
  6. 20 3月, 2015 7 次提交
    • I
      rocksdb: print status error message when (ASSERT|EXPECT)_OK fails · 28bc6de9
      Igor Sugak 提交于
      Summary: Modified rocksdb status assertions ASSERT_OK and EXPECT_OK to print error message from Status::ToString() when failed.
      
      Test Plan: Modify a test to fail status assertions ASSERT_OK and EXPECT_OK and notice an error message that came from Status::ToString()
      
      Reviewers: meyering, sdong, yhchiang, igor
      
      Reviewed By: igor
      
      Subscribers: dhruba, leveldb
      
      Differential Revision: https://reviews.facebook.net/D35469
      28bc6de9
    • I
      rocksdb: Remove #include "util/string_util.h" from util/testharness.h · 9405b5ef
      Igor Sugak 提交于
      Summary:
      1. Manually deleted #include "util/string_util.h" from util/testharness.h
      2.
      ```
      % USE_CLANG=1 make all -j55 -k 2> build.log
      % perl -naF: -E 'say $F[0] if /: error:/' build.log | sort -u | xargs sed -i '/#include "util\/testharness.h"/i #include "util\/string_util.h"'
      ```
      
      Test Plan:
      Make sure make all completes with no errors.
      ```
      % make all -j55
      ```
      
      Reviewers: meyering, igor, sdong
      
      Reviewed By: sdong
      
      Subscribers: dhruba, leveldb
      
      Differential Revision: https://reviews.facebook.net/D35493
      9405b5ef
    • I
      rocksdb: Remove #include "util/random.h" from util/testharness.h · 220d0dff
      Igor Sugak 提交于
      Summary: Cleaning util/testharness.h
      
      Test Plan:
      Make completes with no errors.
      ```
      % make all
      ```
      
      Reviewers: meyering, igor, sdong
      
      Reviewed By: sdong
      
      Subscribers: dhruba, leveldb
      
      Differential Revision: https://reviews.facebook.net/D35487
      220d0dff
    • I
      Don't delete files when column family is dropped · b088c83e
      Igor Canadi 提交于
      Summary:
      To understand the bug read t5943287 and check out the new test in column_family_test (ReadDroppedColumnFamily), iter 0.
      
      RocksDB contract allowes you to read a drop column family as long as there is a live reference. However, since our iteration ignores dropped column families, AddLiveFiles() didn't mark files of a dropped column families as live. So we deleted them.
      
      In this patch I no longer ignore dropped column families in the iteration. I think this behavior was confusing and it also led to this bug. Now if an iterator client wants to ignore dropped column families, he needs to do it explicitly.
      
      Test Plan: Added a new unit test that is failing on master. Unit test succeeds now.
      
      Reviewers: sdong, rven, yhchiang
      
      Reviewed By: yhchiang
      
      Subscribers: dhruba, leveldb
      
      Differential Revision: https://reviews.facebook.net/D32535
      b088c83e
    • I
      rocksdb: initial util/testharness clean up · 17ae3fcb
      Igor Sugak 提交于
      Summary: Deleted some redundant code. More comming.
      
      Test Plan:
      ```lang=bash
      % USE_CLANG=1 make check
      ```
      
      Reviewers: meyering, sdong, igor
      
      Reviewed By: igor
      
      Subscribers: dhruba, leveldb
      
      Differential Revision: https://reviews.facebook.net/D35463
      17ae3fcb
    • F
      [RocksJava] Enhanced Logger comment · 39f4271b
      fyrz 提交于
      Added information about performance penalties using a custom
      logger implementation.
      39f4271b
    • F
      [RocksJava] 32-Bit adjustments · 5615e23d
      fyrz 提交于
      Summary:
      Before this change overflowing size_t values led to a checked Exception.
      
      After that change:
      
      size_t overflows on 32-Bit architecture throw now an IllegalArgumentException,
      which removes the necessity for a developer to catch these Exceptions explicitly.
      
      This is especially an advantage for developers targeting 64-Bit systems because
      it is not necessary anymore to catch exceptions which are never thrown on a 64-Bit
      system.
      
      Test Plan:
      make clean jclean rocksdbjava jtest
      mvn -f rocksjni.pom package
      
      Reviewers: adamretter, yhchiang, ankgup87
      
      Subscribers: dhruba
      
      Differential Revision: https://reviews.facebook.net/D34923
      5615e23d
  7. 19 3月, 2015 1 次提交