Daniel Becker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/18801 )

Change subject: IMPALA-11464: Skip listing staging dirs to avoid failures on 
them
......................................................................


Patch Set 5: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/18801/2/tests/metadata/test_recursive_listing.py
File tests/metadata/test_recursive_listing.py:

http://gerrit.cloudera.org:8080/#/c/18801/2/tests/metadata/test_recursive_listing.py@147
PS2, Line 147:   @SkipIfGCS.variable_listing_times
> Done. BTW, we mentioned it at line 187 as well.
Thanks, I think it's clearer this way, if it is mentioned in the doc comment 
too.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic848e6c8563a1e0bf294cd50167dfc40f66a56cb
Gerrit-Change-Number: 18801
Gerrit-PatchSet: 5
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Daniel Becker <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Xiang Yang <[email protected]>
Gerrit-Comment-Date: Wed, 03 Aug 2022 11:54:32 +0000
Gerrit-HasComments: Yes

Reply via email to