Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10956 )

Change subject: [logging] fix logging init if linking kudu client library
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10956/2/src/kudu/tools/tool_main.cc
File src/kudu/tools/tool_main.cc:

http://gerrit.cloudera.org:8080/#/c/10956/2/src/kudu/tools/tool_main.cc@254
PS2, Line 254:
What motivated the reordering here?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6dcd8d7ccb8a9cafa79674273b95c61eee5baf5a
Gerrit-Change-Number: 10956
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Tue, 17 Jul 2018 05:11:25 +0000
Gerrit-HasComments: Yes

Reply via email to