RocksJava] Fix ColumnFamily tests
Summary: Cleaned up some tests regarding disposal order and tests which were failing when C++ assertions were enabled. Test Plan: - Enable C++ Assertions (remove e.g. -DNDebug in rocksdbjava target) - make rocksdbjava jtest Reviewers: adamretter, ankgup87, yhchiang Subscribers: dhruba Differential Revision: https://reviews.facebook.net/D34569
This commit is contained in:
parent
67533809fd
commit
0d13bbe272
@ -418,9 +418,11 @@ public class DbBenchmark {
|
|||||||
stats_.found_++;
|
stats_.found_++;
|
||||||
stats_.finishedSingleOp(iter.key().length + iter.value().length);
|
stats_.finishedSingleOp(iter.key().length + iter.value().length);
|
||||||
if (isFinished()) {
|
if (isFinished()) {
|
||||||
|
iter.dispose();
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
iter.dispose();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -723,7 +723,7 @@ public class ColumnFamilyTest {
|
|||||||
try {
|
try {
|
||||||
options = new Options().setCreateIfMissing(true);
|
options = new Options().setCreateIfMissing(true);
|
||||||
db = RocksDB.open(options, dbFolder.getRoot().getAbsolutePath());
|
db = RocksDB.open(options, dbFolder.getRoot().getAbsolutePath());
|
||||||
final String simplifiedChinese = "简体字";
|
final String simplifiedChinese = "\u7b80\u4f53\u5b57";
|
||||||
columnFamilyHandle = db.createColumnFamily(
|
columnFamilyHandle = db.createColumnFamily(
|
||||||
new ColumnFamilyDescriptor(simplifiedChinese.getBytes()));
|
new ColumnFamilyDescriptor(simplifiedChinese.getBytes()));
|
||||||
|
|
||||||
|
Loading…
x
Reference in New Issue
Block a user