Fix the valgrind error in newly added unittests for table stats

Summary:

Previous the newly added test called NewBloomFilter without releasing it at the end of the test, which resulted in memory leak and was detected by valgrind.

Test Plan:

Ran valgrind test.

diff --git a/table/table_test.cc b/table/table_test.cc
index 066b7362ecaa758027d17058d92fe9d3519c6f67..5ed8c7070c11470d82cdba26884f8f16bcd9ba72 100644 (file)
--- a/table/table_test.cc
+++ b/table/table_test.cc
@@ -906,7 +906,10 @@ TEST(TableTest, FilterPolicyNameStats) {
   std::vector<std::string> keys;
   KVMap kvmap;
   Options options;
-  options.filter_policy = NewBloomFilterPolicy(10);
+  std::unique_ptr<const FilterPolicy> filter_policy(
+    NewBloomFilterPolicy(10)
+  );
+  options.filter_policy = filter_policy.get();
 
   c.Finish(options, &keys, &kvmap);
   auto& stats = c.table()->GetTableStats();