You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Dan Burkert (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/01/15 21:31:10 UTC

[kudu-CR] Add missing override specifiers

Hello Adar Dembo, Todd Lipcon,

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

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

to review the following change.

Change subject: Add missing override specifiers
......................................................................

Add missing override specifiers

This patch was created using the clang-modernize add-override transform
tool.

Change-Id: I2fe7cef4636462c6a1cb6423567613572a3f923c
---
M src/kudu/codegen/compilation_manager.cc
M src/kudu/consensus/leader_election-test.cc
M src/kudu/tablet/memrowset.cc
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/util/debug/trace_logging.h
M src/kudu/util/metrics-test.cc
M src/kudu/util/striped64.h
7 files changed, 17 insertions(+), 16 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I2fe7cef4636462c6a1cb6423567613572a3f923c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <da...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>