Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4927 to look at the new patch set (#6). Change subject: Add a macro to LOG and return on a non-OK status ...................................................................... Add a macro to LOG and return on a non-OK status We often see the following pattern: Status s = someOperation(); if (!s.ok()) { LOG(ERROR) << "A critical error occurred at blah. Status: " << s.ToString(); return s; } This is cumbersome. This macro allows to do the same the following way: RETURN_NOT_OK_LOG(someOperation(), ERROR, "A critical error occurred at blah."); The macro also prints the status (though at the beginning of the message and not at the end so that we can have arbirarily long debugging information but the status is easy to find). Change-Id: Ibb906e7240d7a289a7822444bc9cbf748a555efb --- M src/kudu/util/status.h 1 file changed, 10 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/27/4927/6 -- To view, visit http://gerrit.cloudera.org:8080/4927 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ibb906e7240d7a289a7822444bc9cbf748a555efb Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>