Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22750 )
Change subject: [benchmarks] fix result status handling ...................................................................... [benchmarks] fix result status handling This patch addresses sloppy result handling of functions/methods that return Status in the following directories * src/kudu/benchmarks * src/kudu/clock * src/kudu/codegen Change-Id: I3d9868da085d3448db876ca93e1ffbbb69a7ef31 Reviewed-on: http://gerrit.cloudera.org:8080/22647 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit e3c9f6b88f486af48ba1a0853e52eaf01bcb6ec6) Reviewed-on: http://gerrit.cloudera.org:8080/22750 --- M src/kudu/benchmarks/tpch/rpc_line_item_dao.cc M src/kudu/clock/builtin_ntp.cc M src/kudu/clock/logical_clock-test.cc M src/kudu/codegen/codegen-test.cc 4 files changed, 10 insertions(+), 7 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/22750 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I3d9868da085d3448db876ca93e1ffbbb69a7ef31 Gerrit-Change-Number: 22750 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
