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)

Reply via email to