impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4987: Fix flaky test test row
Date Tue, 12 Sep 2017 03:58:08 GMT
Alex Behm has posted comments on this change.

Change subject: IMPALA-4987: Fix flaky test

Patch Set 1:


Thanks for fixing this
Commit Message:

Line 9: This patch keeps test_row_availbility from random failure. In this test
Please take a pass to correct wording/grammar. For example, it should be:

"This patch keeps test_rows_availability from randomly failing."

More such examples below.

Line 10: the time interval between 'Rows available' event and the previous event
the 'Rows available' timeline event

Line 11: in runtime profile is measured in order to make sure that rows become
in the runtime profile

Line 12: available after a specific amount of time. This is not correct since
Instead of 'This' try to rephrase/repeat what 'This' refers to for clarity, e.g. This measurement
is not correct ...

Line 13: the previous event is that the coordinator finishes sending query to
finished sending the query to the backends

Line 14: backends, which means the execution on backend might have already
on some backends might have already started

Line 15: started. This patch tracks another event "ready to start" as the
"Read to start"

Line 17: query to backends after this event so the time check should always pass.
the query to backends
File tests/query_test/

Line 81:         row_avail_time_ms = self.__parse_time_ms(self.__find_time(line))
rows_avail_time_ms (not just one row is available)

Line 92:         "'ready to start' event.\nExpected the event to be marked no earlier than
'Ready to start'

Line 93:         "%sms after the 'ready to start' event.\nQuery: %s"\
'Ready to start'

Line 99:     """Find event time point in a line from runtime timeline."""
from the runtime profile timeline

Line 100:     # Example line: "- Rows available: 3s311ms (2s300ms)"
Example should include what this function returns

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I96142f1868a26426cbc34aa9a0e0a56979df66c3
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tianyi Wang <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-HasComments: Yes

View raw message