Dan Burkert has submitted this change and it was merged.

Change subject: util: add Status::AndThen combinator
......................................................................


util: add Status::AndThen combinator

The AndThen combinator makes it easier to chain together
Status-returning operations when failure recovery is something other
than returning early. RETURN_NOT_OK already handles the return-early
case elegantly.

Also tweaks Status::CloneAndPrepend and Status::CloneAndAppend to be
usable with OK status values.

Change-Id: Ia6e1a79ff95406825d4238a8157d242252562805
Reviewed-on: http://gerrit.cloudera.org:8080/7399
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/util/status-test.cc
M src/kudu/util/status.cc
M src/kudu/util/status.h
3 files changed, 60 insertions(+), 2 deletions(-)

Approvals:
  Alexey Serbin: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia6e1a79ff95406825d4238a8157d242252562805
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to