Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20261 )
Change subject: [tests] a small clean-up on create-table-itest.cc ...................................................................... [tests] a small clean-up on create-table-itest.cc This patch updates the code in create-table-itest.cc to fix the compilation warning on improper handling of the return status of the ExternalTabletServer::Restart() method. In addition, it contains other unsorted improvements (mostly code style-related) on the test code. This is a follow-up to a5648f39b407c62b70ca0ce6fefd0fdab9533daf. Change-Id: I7508f1bf0c286098c673019731c8551efd2d8d78 Reviewed-on: http://gerrit.cloudera.org:8080/20261 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/integration-tests/create-table-itest.cc 1 file changed, 37 insertions(+), 24 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/20261 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7508f1bf0c286098c673019731c8551efd2d8d78 Gerrit-Change-Number: 20261 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: Wang Xixu <[email protected]>
