Hello Dan Burkert, Alexey Serbin,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/7826

to review the following change.

Change subject: catalog_manager: don't log deleted tables/tablets at startup
......................................................................

catalog_manager: don't log deleted tables/tablets at startup

On long-lived clusters this list can get quite long.

While I was in the area, I made some other cosmetic improvements:
- Applying strings::Substitute() to perforated LOG statements.
- Replacing 'OVERRIDE' with 'override'.
- Removing 'virtual' from overriden methods.
- Removing unnecessary std:: prefixes.
- A few other miscellaneous tweaks.

Change-Id: I557ebfe9d3ce1663e7f7c189f0087bdd29a09306
---
M src/kudu/master/catalog_manager.cc
M src/kudu/master/sys_catalog.h
2 files changed, 191 insertions(+), 205 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/26/7826/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7826
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I557ebfe9d3ce1663e7f7c189f0087bdd29a09306
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>

Reply via email to