Hello Yida Wu, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22724
to look at the new patch set (#3).
Change subject: IMPALA-13925: Deflake TestFrontendConnectionLimit
......................................................................
IMPALA-13925: Deflake TestFrontendConnectionLimit
TestFrontendConnectionLimit is flaky after IMPALA-13907. The test
randomly pick one impalad and create multiple client to connect to that
impalad. However, the test also limit --fe_service_threads=1. If the
random impalad picked is the first impalad, any new client might not
able to connect because the default ImpalaTestSuite.hs2_client is still
connect to the first impalad.
This patch attempt to deflake the test by skip creating all default
impala clients. Added classmethod need_default_clients() that is
overrideable by test classes that wish to skip default client creation.
Testing:
Manually loop the tests for 20 times and pass all of them. Before this
patch, the test can get stuck if looped less than that.
Change-Id: I53c6760f5e3734397746b5a228345c9df38eabcb
---
M tests/common/impala_test_suite.py
M tests/custom_cluster/test_frontend_connection_limit.py
2 files changed, 42 insertions(+), 26 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/24/22724/3
--
To view, visit http://gerrit.cloudera.org:8080/22724
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I53c6760f5e3734397746b5a228345c9df38eabcb
Gerrit-Change-Number: 22724
Gerrit-PatchSet: 3
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Yida Wu <[email protected]>
