Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/9752 )
Change subject: KUDU-972. Cache should track the memory overhead ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/9752/1/src/kudu/cfile/block_cache-test.cc File src/kudu/cfile/block_cache-test.cc: http://gerrit.cloudera.org:8080/#/c/9752/1/src/kudu/cfile/block_cache-test.cc@43 PS1, Line 43: MemTracker::FindTracker("block_cache-sharded_lru_cache", &mem_tracker); > Could just ASSERT_TRUE() on the return result. Done -- To view, visit http://gerrit.cloudera.org:8080/9752 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I6cc69004dccb1cc372fdedbac321154e46a92263 Gerrit-Change-Number: 9752 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 22 Mar 2018 23:43:57 +0000 Gerrit-HasComments: Yes
