Surya Hebbar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22482 )

Change subject: IMPALA-13751: Fix runtime-profile-test failure since 
IMPALA-13304
......................................................................


Patch Set 8:

Please let me know, if I am missing something.

The actual fix without any changes would have just been 5-10 lines. The ones 
mentioned in the causes.

This test already provides much better coverage.

The reason for randomized tests was to cover sorting and alignment with 
possible missing events, such tests would not be relevant otherwise.


--
To view, visit http://gerrit.cloudera.org:8080/22482
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If04744e215cac79f255b3d73c3e91e873c13749a
Gerrit-Change-Number: 22482
Gerrit-PatchSet: 8
Gerrit-Owner: Surya Hebbar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Surya Hebbar <[email protected]>
Gerrit-Comment-Date: Wed, 19 Feb 2025 23:16:47 +0000
Gerrit-HasComments: No

Reply via email to