Yifan Zhang has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/18947 )
Change subject: [tools] mark long-running scenarios in kudu-tool-test ...................................................................... [tools] mark long-running scenarios in kudu-tool-test I noticed that it takes very long time for kudu-tool-test to complete. It turned out the test suite contains many scenarios that run with HMS integration enabled and it takes a long time to start up the HMS process. This patch marks the scenarios that tend to run over 3 seconds as slow, so they are skipped unless the KUDU_ALLOW_SLOW_TESTS environment variable is set to true/yes/1. The run time of the kudu-tool-test with this patch looks like the following on a contemporary Linux machine: Without KUDU_ALLOW_SLOW_TESTS: [==========] 128 tests from 8 test suites ran. (86355 ms total) [ PASSED ] 88 tests. [ SKIPPED ] 40 tests, listed below: ... With KUDU_ALLOW_SLOW_TESTS=1: [==========] 128 tests from 8 test suites ran. (842684 ms total) [ PASSED ] 128 tests. Change-Id: I1893741fada185d819128396c88f4ac99f43ed40 Reviewed-on: http://gerrit.cloudera.org:8080/18947 Reviewed-by: Mahesh Reddy <[email protected]> Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Yifan Zhang <[email protected]> --- M src/kudu/tools/kudu-tool-test.cc 1 file changed, 26 insertions(+), 0 deletions(-) Approvals: Mahesh Reddy: Looks good to me, but someone else must approve Alexey Serbin: Verified Yifan Zhang: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/18947 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1893741fada185d819128396c88f4ac99f43ed40 Gerrit-Change-Number: 18947 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
