]> git-server-git.apps.pok.os.sepia.ceph.com Git - rocksdb.git/commit
Fix the memory leak with pinned partitioned filters
authorMaysam Yabandeh <myabandeh@fb.com>
Mon, 9 Apr 2018 23:17:15 +0000 (16:17 -0700)
committerFacebook Github Bot <facebook-github-bot@users.noreply.github.com>
Mon, 9 Apr 2018 23:28:19 +0000 (16:28 -0700)
commitd2bcd7611f0881b21ae2c77100e3e055bf01ae5f
tree9035d2d437109d7d10cb2691521f5d5543744b65
parent65fe8d6cd6083dbef73af8ff273fb570236331f6
Fix the memory leak with pinned partitioned filters

Summary:
The existing unit test did not set the level so the check for pinned partitioned filter/index being properly released from the block cache was not properly exercised as they only take effect in level 0. As a result a memory leak in pinned partitioned filters was hidden. The patch fix the test as well as the bug.
Closes https://github.com/facebook/rocksdb/pull/3692

Differential Revision: D7559763

Pulled By: maysamyabandeh

fbshipit-source-id: 55eff274945838af983c764a7d71e8daff092e4a
HISTORY.md
table/partitioned_filter_block.cc
table/partitioned_filter_block.h
table/table_test.cc