You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by bb...@apache.org on 2019/05/04 14:53:04 UTC
[kafka] branch trunk updated: Should close filter in
RocksDBStoreTest as well (#6676)
This is an automated email from the ASF dual-hosted git repository.
bbejeck pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git
The following commit(s) were added to refs/heads/trunk by this push:
new c85152f Should close filter in RocksDBStoreTest as well (#6676)
c85152f is described below
commit c85152f5b96a0904b7b503f8edb1485e41daca05
Author: A. Sophie Blee-Goldman <so...@confluent.io>
AuthorDate: Sat May 4 07:52:53 2019 -0700
Should close filter in RocksDBStoreTest as well (#6676)
Forgot to also close the filter in RocksDBStoreTest in time.
Reviewers: Bill Bejeck <bb...@gmail.com>
---
.../org/apache/kafka/streams/state/internals/RocksDBStoreTest.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/streams/src/test/java/org/apache/kafka/streams/state/internals/RocksDBStoreTest.java b/streams/src/test/java/org/apache/kafka/streams/state/internals/RocksDBStoreTest.java
index db458eb..c9dde77 100644
--- a/streams/src/test/java/org/apache/kafka/streams/state/internals/RocksDBStoreTest.java
+++ b/streams/src/test/java/org/apache/kafka/streams/state/internals/RocksDBStoreTest.java
@@ -67,6 +67,7 @@ public class RocksDBStoreTest {
InternalMockProcessorContext context;
RocksDBStore rocksDBStore;
+ private static BloomFilter filter;
@Before
public void setUp() {
@@ -78,6 +79,7 @@ public class RocksDBStoreTest {
Serdes.String(),
Serdes.String(),
new StreamsConfig(props));
+ filter = new BloomFilter();
}
RocksDBStore getRocksDBStore() {
@@ -86,6 +88,7 @@ public class RocksDBStoreTest {
@After
public void tearDown() {
+ filter.close();
rocksDBStore.close();
}
@@ -495,7 +498,7 @@ public class RocksDBStoreTest {
tableConfig.setBlockCacheSize(50 * 1024 * 1024L);
tableConfig.setBlockSize(4096L);
if (enableBloomFilters) {
- tableConfig.setFilter(new BloomFilter());
+ tableConfig.setFilter(filter);
options.optimizeFiltersForHits();
bloomFiltersSet = true;
} else {