Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/14337 )
Change subject: IMPALA-8926, IMPALA-8989: Fix flaky result spooling tests ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14337/1/tests/custom_cluster/test_admission_controller.py File tests/custom_cluster/test_admission_controller.py: http://gerrit.cloudera.org:8080/#/c/14337/1/tests/custom_cluster/test_admission_controller.py@1337 PS1, Line 1337: sleep(30) # Wait for the 'lineitem' scan to complete > The goal of the test is to actually validate that NumCompletedBackends is i assert_eventually still has a timeout itself so it will suffer the same timing issue in the worst case. Given that this test checks that NumCompletedBackends is incremented by one, isn't it more precise to check that all F01 reached "FINISHED" states by parsing the json output from the debug webpage ? http://<host-name>:25000/query_finstances?query_id=<query-id>&json A proxy for the above is to check the number of in-flight fragments on all nodes. This of course bakes a lot of assumption into the test about the generated plan for the given query. -- To view, visit http://gerrit.cloudera.org:8080/14337 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7ea6bf3d84f174745c8a0b1e0f2b55ce05ee618b Gerrit-Change-Number: 14337 Gerrit-PatchSet: 1 Gerrit-Owner: Sahil Takiar <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Sahil Takiar <[email protected]> Gerrit-Comment-Date: Wed, 02 Oct 2019 17:45:26 +0000 Gerrit-HasComments: Yes
