Hello Quanlong Huang, Zoltan Borok-Nagy, Joe McDonnell, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/21162 to look at the new patch set (#3). Change subject: IMPALA-12898: Tidy up test dimensions of test_scanner.py ...................................................................... IMPALA-12898: Tidy up test dimensions of test_scanner.py This patch tidies up the test dimensions of test_scanner.py. 'exec_option' initialization is moved to add_test_dimensions() method as much as possible. It ensures correct permutation and execution of test cases. After this patch, the total collected tests of test_scanner.py is 1242 for core/pairwise exploration and 7514 for exhaustive exploration. Before, they were 794 and 11864 accordingly. The increase in test count after refactoring with core exploration is because exec option dimensions are now permuted correctly along with other default exec option dimensions. The reduction in exhaustive exploration is due to a reduction in the overall dimension to permute and a reduction in test skipping (the test was run, but only called pytest.skip()). Testing: - Pass query_test/test_scanners.py in exhaustive exploration. Change-Id: I5efd2b483338fb55b958d8e1a0acf6b365f8093e --- M tests/query_test/test_scanners.py 1 file changed, 82 insertions(+), 83 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/62/21162/3 -- To view, visit http://gerrit.cloudera.org:8080/21162 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I5efd2b483338fb55b958d8e1a0acf6b365f8093e Gerrit-Change-Number: 21162 Gerrit-PatchSet: 3 Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com> Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com> Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>