Csaba Ringhofer has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/23319 )

Change subject: IMPALA-14333: Run impala-py.test using Python3
......................................................................


Patch Set 12:

I have a patch that replaces the pairwise test vector generation with something 
that actually covers all pairs:
https://gerrit.cloudera.org/#/c/23342/

The first version bumped the number of customer cluster/EE tests from 11K to 
17K in exhaustive runs and revealed several issues found by your patch. I 
updated it to fix some failures and reduce the number of tests a bit (it is 
still running)

I copied some of the fixes from here to that patch, while I did a few things 
differently (removing some file format + compression pairs)

I think that the test fixes for the issues revealed by the different test 
vector generations should be merged in a separate commit than the Py2->3 
change, as it seems just an accident the Python version change led to more test 
vectors.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I401a93b6cc7bcd17f41d24e7a310e0c882a550d4
Gerrit-Change-Number: 23319
Gerrit-PatchSet: 12
Gerrit-Owner: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Laszlo Gaal <laszlo.g...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Comment-Date: Tue, 26 Aug 2025 11:57:20 +0000
Gerrit-HasComments: No

Reply via email to