Internal Jenkins has submitted this change and it was merged.

Change subject: Make test_low_mem_limit_low_selectivity_scan faster
......................................................................


Make test_low_mem_limit_low_selectivity_scan faster

The took ~7 mins to run on my machine. It was spending about half its
time re-creating the same data set over and over. This may also help
with the test failures during continuous integration.

All that needed to be done was to change the setup/teardown from method
level to class level. This should have no affect on the test coverage.

Change-Id: Ic874a7e9c8305bef3e16df03a807493f069e9265
Reviewed-on: http://gerrit.cloudera.org:8080/2766
Reviewed-by: Alex Behm <[email protected]>
Tested-by: Internal Jenkins
---
M tests/query_test/test_kudu.py
1 file changed, 29 insertions(+), 25 deletions(-)

Approvals:
  Internal Jenkins: Verified
  Alex Behm: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic874a7e9c8305bef3e16df03a807493f069e9265
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Casey Ching <[email protected]>
Gerrit-Reviewer: Internal Jenkins
Gerrit-Reviewer: Michael Brown <[email protected]>

Reply via email to