Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/24987
Change subject: IMPALA-15137: Fix some flaky TestAdmissionControllerWithACService tests ...................................................................... IMPALA-15137: Fix some flaky TestAdmissionControllerWithACService tests _execute_and_collect_profiles() started the queries in quick succession, expecting them to reach the admission queue in the same order - the fetching of results happens in the original order and was not prepared for out of order admission (the next query can't be admitted before the results of the previous are fetched as result spooling is disabled for these queries). As ExecuteStatement RPC waits only for planning to finish and admission is done on a different thread, a query requested later can actually reach the admission queue earlier. This is more likely in tests with admissiond where the gap between planning and admission is longer, leading to frequent test failures in the ACService case. The fix is to serialize admission by waiting for a query to reach the admission queue (admitted, queued or rejected) before starting the next one. This adds some delay to the function (worst case around 100ms per query start when admissiond is used), but this seems small enough to no alter the semantics of the calling tests. Assisted-by: Claude Opus 5 (Claude Code) Change-Id: I6e0bc611adee662f1e9ce2722ac843dc11dcb3ad --- M tests/custom_cluster/test_admission_controller.py 1 file changed, 15 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/87/24987/1 -- To view, visit http://gerrit.cloudera.org:8080/24987 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I6e0bc611adee662f1e9ce2722ac843dc11dcb3ad Gerrit-Change-Number: 24987 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <[email protected]>
