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

Change subject: data_dirs: fix logging message
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/8999/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/8999/1//COMMIT_MSG@10
PS1, Line 10:
Nit: got an extra space here


http://gerrit.cloudera.org:8080/#/c/8999/1/src/kudu/fs/data_dirs.cc
File src/kudu/fs/data_dirs.cc:

http://gerrit.cloudera.org:8080/#/c/8999/1/src/kudu/fs/data_dirs.cc@834
PS1, Line 834:       LOG(INFO) << Substitute(msg);
Why INFO and not WARNING?



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8135e0ad2411524fcb74b3e349bea7a5828a4237
Gerrit-Change-Number: 8999
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Wed, 10 Jan 2018 22:29:30 +0000
Gerrit-HasComments: Yes

Reply via email to