]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/log
rocksdb.git
9 years agocmake: drop "-march=native" from CXX_FLAGS wip-no-march-native
Kefu Chai [Fri, 28 Oct 2016 10:47:54 +0000 (18:47 +0800)]
cmake: drop "-march=native" from CXX_FLAGS

this breaks the cross-compiling, and we can not assume that the building
machine and the target machine share the same CPU spec.

Signed-off-by: Kefu Chai <kchai@redhat.com>
9 years agoMakefile: generate util/build_version.cc from .in file (#1384)
Kefu Chai [Tue, 25 Oct 2016 18:31:39 +0000 (02:31 +0800)]
Makefile: generate util/build_version.cc from .in file (#1384)

* util/build_verion.cc.in: add this file, so cmake and make can share the
  template file for generating util/build_version.cc.
* CMakeLists.txt: also, cmake v2.8.11 does not support file(GENERATE ...),
  so we are using configure_file() for creating build_version.cc.
* Makefile: use util/build_verion.cc.in for creating build_version.cc.

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
9 years agoDisable DBTest.RepeatedWritesToSameKey (#1420)
Siying Dong [Tue, 25 Oct 2016 17:23:50 +0000 (10:23 -0700)]
Disable DBTest.RepeatedWritesToSameKey (#1420)

Summary:
The verification condition of the test DBTest.RepeatedWritesToSameKey doesn't hold anymore after 3ce3bb3da2486c2c18a332128dda7c05a91abb85.
Disable the test for now before we find a way to replace it.

Test Plan: Run the test and make sure it is disabled.

9 years agoOptionChangeMigration() to support FIFO compaction
sdong [Mon, 24 Oct 2016 20:16:54 +0000 (13:16 -0700)]
OptionChangeMigration() to support FIFO compaction

Summary: OptionChangeMigration() to support FIFO compaction. If the DB before migration is using FIFO compaction, nothing should be done. If the desitnation option is FIFO options, compact to one single L0 file if the source has more than one levels.

Test Plan: Run option_change_migration_test

Reviewers: andrewkr, IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: leveldb, andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D65289

9 years agoChanging the legocastle run to use valgrind_test instead of _check
Anirban Rahut [Mon, 24 Oct 2016 23:23:19 +0000 (16:23 -0700)]
Changing the legocastle run to use valgrind_test instead of _check

Summary:
valgrind_test is the correct way to run valgrind tests.
this is becasue we need to force DISABLE_JEMALLOC

Test Plan: Running sandcastle and contrun

Reviewers: IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D65451

9 years agorevert Prev() in MergingIterator to use previous code in non-prefix-seek mode
Aaron Gao [Mon, 24 Oct 2016 20:13:01 +0000 (13:13 -0700)]
revert Prev() in MergingIterator to use previous code in non-prefix-seek mode

Summary: Siying suggested to keep old code for normal mode prev() for safety

Test Plan: make check -j64

Reviewers: yiwu, andrewkr, sdong

Reviewed By: sdong

Subscribers: andrewkr, dhruba, leveldb

Differential Revision: https://reviews.facebook.net/D65439

9 years agoDBSSTTest.RateLimitedDelete: not to use real clock
sdong [Tue, 18 Oct 2016 05:22:48 +0000 (22:22 -0700)]
DBSSTTest.RateLimitedDelete: not to use real clock

Summary: Using real clock causes failures of DBSSTTest.RateLimitedDelete in some cases. Turn away from the real time. Use fake time instead.

Test Plan: Run the tests and all existing tests.

Reviewers: yiwu, IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: leveldb, andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D65145

9 years agoFix a bug that may cause a deleted row to appear again
sdong [Fri, 21 Oct 2016 19:50:01 +0000 (12:50 -0700)]
Fix a bug that may cause a deleted row to appear again

Summary:
The previous fix of reappearing of a deleted row 0ce258f9b37c8661ea326039372bef8f185615ef missed a corner case, which can be reproduced using test CompactionPickerTest.OverlappingUserKeys7. Consider such an example:

input level file: 1[B E] 2[F H]
output level file: 3[A C] 4[D I] 5[I K]

First file 2 is picked, which overlaps to file 4. 4 expands to 5. Now the all range is [D K] with 2 output level files. When we try to expand that, [D K] overlaps with file 1 and 2 in the input level, and 1 and 2 overlaps with 3 and 4 in the output level. So we end up with picking 3 and 4 in the output level. Without expanding, it also has 2 files, so we determine the output level doesn't change, although they are the different two files.

The fix is to expand the output level files after we picked 3 and 4. In that case, there will be three output level files so we will abort the expanding.

I also added two unit tests related to marked_for_compaction and being_compacted. They have been passing though.

Test Plan: Run the new unit test, as well as all other tests.

Reviewers: andrewkr, IslamAbdelRahman

Reviewed By: IslamAbdelRahman

Subscribers: yoshinorim, leveldb, andrewkr, dhruba

Differential Revision: https://reviews.facebook.net/D65373

9 years agoFix integer overflow in GetL0ThresholdSpeedupCompaction (#1378)
Edouard A [Mon, 24 Oct 2016 01:43:29 +0000 (18:43 -0700)]
Fix integer overflow in GetL0ThresholdSpeedupCompaction (#1378)

9 years agoFix a bug that mistakenly disable regression_test.sh to update commit (#1415)
Yueh-Hsuan Chiang [Sat, 22 Oct 2016 00:26:24 +0000 (17:26 -0700)]
Fix a bug that mistakenly disable regression_test.sh to update commit (#1415)

Summary:
Fix a bug that mistakenly disable regression_test.sh to update commit

Test Plan:
regression_test.sh