]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/log
rocksdb.git
10 years agoRelease 4.3.1 rocksdb-4.3 rocksdb-4.3.1 v4.3 v4.3.1
Islam AbdelRahman [Wed, 27 Jan 2016 03:07:19 +0000 (19:07 -0800)]
Release 4.3.1

10 years agoFix BlockBasedTableTest.NoopTransformSeek failure
Islam AbdelRahman [Thu, 7 Jan 2016 17:48:29 +0000 (09:48 -0800)]
Fix BlockBasedTableTest.NoopTransformSeek failure

Summary:
table_test is failing because we are creating a temp InternalComparator

14:27:28 [ RUN      ] BlockBasedTableTest.NoopTransformSeek
14:27:28 pure virtual method called
14:27:28 terminate called without an active exception
14:27:28 /bin/sh: line 7: 2346261 Aborted                 (core dumped) ./$t

Test Plan: make table_test -j64 && ./table_test --gtest_filter="BlockBasedTableTest.NoopTransformSeek"

Reviewers: igor, sdong, anthony, rven

Reviewed By: rven

Subscribers: dhruba

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

10 years agoFix issue in Iterator::Seek when using Block based filter block with prefix_extractor
Islam AbdelRahman [Tue, 26 Jan 2016 22:47:42 +0000 (14:47 -0800)]
Fix issue in Iterator::Seek when using Block based filter block with prefix_extractor

Summary: Similar to D53385 we need to check InDomain before checking the filter block.

Test Plan: unit tests

Reviewers: yhchiang, rven, sdong

Reviewed By: sdong

Subscribers: dhruba

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

10 years agoFix bug in block based tables with full filter block and prefix_extractor
Islam AbdelRahman [Tue, 26 Jan 2016 19:07:08 +0000 (11:07 -0800)]
Fix bug in block based tables with full filter block and prefix_extractor

Summary:
Right now when we are creating a BlockBasedTable with fill filter block
we add to the filter all the prefixes that are InDomain() based on the prefix_extractor

the problem is that when we read a key from the file, we check the filter block for the prefix whether or not it's InDomain()

Test Plan: unit tests

Reviewers: yhchiang, rven, anthony, kradhakrishnan, sdong

Reviewed By: sdong

Subscribers: dhruba

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

10 years agoFix index seeking in BlockTableReader::PrefixMayMatch.
Peter Mattis [Wed, 6 Jan 2016 19:46:32 +0000 (14:46 -0500)]
Fix index seeking in BlockTableReader::PrefixMayMatch.

PrefixMayMatch previously seeked in the prefix index using an internal
key with a sequence number of 0. This would cause the prefix index seek
to fall off the end if the last key in the index had a user-key greater
than or equal to the key being looked for. Falling off the end of the
index in turn results in PrefixMayMatch returning false if the index is
in memory.

10 years agoRevert "db_bench: --soft_pending_compaction_bytes_limit should set options.soft_pendi...
sdong [Wed, 23 Dec 2015 17:40:37 +0000 (09:40 -0800)]
Revert "db_bench: --soft_pending_compaction_bytes_limit should set options.soft_pending_compaction_bytes_limit"

This reverts commit c726b0fdc1dd9ed178f0fb740d30f72516b7106a.

Bad backporting

10 years agodb_bench: --soft_pending_compaction_bytes_limit should set options.soft_pending_compa...
sdong [Fri, 18 Dec 2015 02:10:48 +0000 (18:10 -0800)]
db_bench: --soft_pending_compaction_bytes_limit should set options.soft_pending_compaction_bytes_limit

Summary: Fix a bug that options.soft_pending_compaction_bytes_limit is not actually set with --soft_pending_compaction_bytes_limit

Test Plan: Run db_bench with this parameter and make sure the parameter is set correctly.

Subscribers: leveldb, dhruba

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

10 years agoFixed the valgrind error in ColumnFamilyTest::CreateAndDropRace
Yueh-Hsuan Chiang [Thu, 10 Dec 2015 19:53:53 +0000 (11:53 -0800)]
Fixed the valgrind error in ColumnFamilyTest::CreateAndDropRace

Summary: Fixed the valgrind error in ColumnFamilyTest::CreateAndDropRace

Test Plan: valgrind --error-exitcode=2 --leak-check=full ./column_family_test

Reviewers: kradhakrishnan, rven, anthony, IslamAbdelRahman, sdong

Reviewed By: sdong

Subscribers: dhruba, leveldb

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

10 years agoRevert "Support marking snapshots for write-conflict checking - Take 2"
krad [Wed, 9 Dec 2015 01:53:20 +0000 (17:53 -0800)]
Revert "Support marking snapshots for write-conflict checking - Take 2"

This reverts commit e5c5f23814a995797d64fa3c2bc65dd0a1d27aaa.

10 years agoUpdating HISTORY.md
krad [Wed, 9 Dec 2015 01:45:25 +0000 (17:45 -0800)]
Updating HISTORY.md

Summary: Added 4.3.0 version

Test Plan:

Reviewers:

CC: leveldb@
Task ID: #9298965

Blame Rev: