Hello Tidy Bot, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21018 to look at the new patch set (#2). Change subject: KUDU-613: Cleanup of cache code ...................................................................... KUDU-613: Cleanup of cache code This patch moves some classes out of the anonymous namespace and into the headers of the cache and nvm_cache files. These classes will be used by the new SLRU cache. This patch also templatizes the HandleDeleter and the PendingHandleDeleter class as it will also be used by the SLRU cache. Change-Id: I506d4577c0ae873b01d7fa4f53846d6fd0f664cf --- M src/kudu/cfile/block_cache.h M src/kudu/master/table_locations_cache.cc M src/kudu/util/cache.cc M src/kudu/util/cache.h M src/kudu/util/file_cache.cc M src/kudu/util/nvm_cache.cc M src/kudu/util/nvm_cache.h M src/kudu/util/ttl_cache.h 8 files changed, 297 insertions(+), 286 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/18/21018/2 -- To view, visit http://gerrit.cloudera.org:8080/21018 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I506d4577c0ae873b01d7fa4f53846d6fd0f664cf Gerrit-Change-Number: 21018 Gerrit-PatchSet: 2 Gerrit-Owner: Mahesh Reddy <mre...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)