Hello Jason Fehr, Zoltan Borok-Nagy, Michael Smith, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22658
to look at the new patch set (#2).
Change subject: IMPALA-13889: Add assertion in TestWebPage::test_query_progress
......................................................................
IMPALA-13889: Add assertion in TestWebPage::test_query_progress
test_query_progress has been flaky for not finding the expected
"query_progress" value. This patch attempt to find more clues about the
test failure by:
- Matching against "query_id" instead of "stmt".
- Asserting other JSON values.
- Using fresh hs2 client in __run_query_and_get_debug_page to avoid
query option leak between tests.
Testing:
- In ASAN build, loop and pass test_web_pages.py 10 times.
Change-Id: I5b8bb768bb513c3055a83106511f5a6134a674a6
---
M tests/webserver/test_web_pages.py
1 file changed, 33 insertions(+), 28 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/58/22658/2
--
To view, visit http://gerrit.cloudera.org:8080/22658
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I5b8bb768bb513c3055a83106511f5a6134a674a6
Gerrit-Change-Number: 22658
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>
