Alexey Serbin has posted comments on this change.

Change subject: [security] add WARN_UNUSED_RESULT where possible
......................................................................


Patch Set 3: Code-Review+2 Verified+1

This looks great.

Build: it's a known flakey of 
MasterReplicationTest.TestTimeoutWhenAllMastersAreDown.  The patch to fix it in 
review: https://gerrit.cloudera.org/#/c/6134/

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie8f5a0aea10709640e6c15c63d28b44e5de0ec6d
Gerrit-PatchSet: 3
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to