Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8620 )
Change subject: tablet: remove ignore_result in ApplyRowOperation ...................................................................... tablet: remove ignore_result in ApplyRowOperation ApplyRowOperation() previously ignored the result it returned. This patch updates this to check for errors that we expect, rather than a blanket ignorance of all errors. Unexpected Statuses are returned. Change-Id: I54ea74f5ee10733b2d7b13b1971f9177dddb50cb Reviewed-on: http://gerrit.cloudera.org:8080/8620 Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tablet/tablet.cc 1 file changed, 11 insertions(+), 5 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8620 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I54ea74f5ee10733b2d7b13b1971f9177dddb50cb Gerrit-Change-Number: 8620 Gerrit-PatchSet: 5 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
