Will Berkeley has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9881 )

Change subject: [tools] ksck improvements [1/n]: Eliminate KsckMaster class
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/9881/1/src/kudu/tools/ksck_remote.h
File src/kudu/tools/ksck_remote.h:

http://gerrit.cloudera.org:8080/#/c/9881/1/src/kudu/tools/ksck_remote.h@113
PS1, Line 113:       : master_addresses_(std::move(master_addresses)),
> warning: initializer for base class 'kudu::tools::KsckCluster' is redundant
Done



--
To view, visit http://gerrit.cloudera.org:8080/9881
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I21f9e244b6ba10e11327c744abc1ac72a9b2ab1c
Gerrit-Change-Number: 9881
Gerrit-PatchSet: 2
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Sun, 01 Apr 2018 22:34:18 +0000
Gerrit-HasComments: Yes

Reply via email to