Riza Suminto has posted comments on this change. ( http://gerrit.cloudera.org:8080/23319 )
Change subject: IMPALA-14333: [part 1] Make all pytest compatible with Python3 ...................................................................... Patch Set 2: > Patch Set 2: > > Build Failed > > https://jenkins.impala.io/job/gerrit-code-review-checks/20217/ : Initial code > review checks failed. See linked job for details on the failure. flake8 errors out here: 14:51:16 Traceback (most recent call last): 14:51:16 File "/var/lib/jenkins/workspace/gerrit-auto-critic-test/Impala/bin/jenkins/critique-gerrit-review.py", line 490, in <module> 14:51:16 comments = get_flake8_comments(base_revision, revision) 14:51:16 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 14:51:16 File "/var/lib/jenkins/workspace/gerrit-auto-critic-test/Impala/bin/jenkins/critique-gerrit-review.py", line 168, in get_flake8_comments 14:51:16 raise Exception("Did not expect flake8-diff to write to stderr:\n{0}".format(stderr)) 14:51:16 Exception: Did not expect flake8-diff to write to stderr: 14:51:16 2025-08-20 21:51:14,780 flake8diff.utils ERROR b"<unknown>:890: SyntaxWarning: invalid escape sequence '\\$'\n<unknown>:898: SyntaxWarning: invalid escape sequence '\\$'\n<unknown>:1091: SyntaxWarning: invalid escape sequence '\\;'\n" Not sure what happen, but local run in my machine does not produce any flake8 warnings. -- 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: 2 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Wed, 20 Aug 2025 23:26:29 +0000 Gerrit-HasComments: No
