Dinesh Bhat has posted comments on this change.

Change subject: KUDU-78. Fix pb_util functions which return bool to return 
Status
......................................................................


Patch Set 1:

(1 comment)

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

Line 9: As suggested in ticket, I've refactored four pb_util functions to 
return Status instead of bool.
Nit: IIRC typically we keep commit message line wrapped to 80 chars or below 
unless we have a crash output or some LOG output as part of commit message.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib760793f6e6da3e357573e525f47b32c79472468
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Maxim Smyatkin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dinesh Bhat <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to