Hello Joe McDonnell, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12589

to look at the new patch set (#2).

Change subject: IMPALA-8249: End-to-end test framework doesn't read aggregated 
counters properly
......................................................................

IMPALA-8249: End-to-end test framework doesn't read aggregated counters properly

Updated compute_aggregation() function to not read the pretty-printed
value from the runtime profile, but the accurate value which is at
the end of the line in parenthesis, e.g.:

RowsReturned: 2.14M (2142543)

The old regex tried to parse '2.14M' with 'd+', which resulted in '2'
instead of 2142543.

I tested the change manually.

Change-Id: I2a6fc0d3f7cbaa87aa848cdafffad21fb1514930
---
M tests/common/test_result_verifier.py
1 file changed, 10 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/89/12589/2
--
To view, visit http://gerrit.cloudera.org:8080/12589
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I2a6fc0d3f7cbaa87aa848cdafffad21fb1514930
Gerrit-Change-Number: 12589
Gerrit-PatchSet: 2
Gerrit-Owner: Zoltan Borok-Nagy <borokna...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>

Reply via email to