Joe McDonnell has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24333 )

Change subject: IMPALA-15026: Update Python infra dependencies for CVEs
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/24333/1/infra/python/deps/requirements.txt
File infra/python/deps/requirements.txt:

http://gerrit.cloudera.org:8080/#/c/24333/1/infra/python/deps/requirements.txt@48
PS1, Line 48: pytest == 9.0.3
I tried this a while back and ran into issues with how we use HS2TestSuite. I 
split it off of the python 3.11 upgrade to avoid complication. I have a 
prototype here that I could update: https://gerrit.cloudera.org/#/c/23981/


http://gerrit.cloudera.org:8080/#/c/24333/1/infra/python/deps/requirements.txt@74
PS1, Line 74: sqlparse == 0.5.5
Newer versions of sqlparse have a bug that doesn't parse some of our SQLs 
properly. It breaks dataload, which relies on it splitting up SQL files into 
individual SQLs. I think it was related to escaped backslash. There is an issue 
posted on the sqlparse project about it: 
https://github.com/andialbrecht/sqlparse/issues/814 and there are a couple PRs 
trying to address it that haven't been merged. I was hoping sqlparse might put 
out a 0.5.6 with the fixes, but it has been a while.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7d2811cec99dd21d201faaae413b37e44f5f051e
Gerrit-Change-Number: 24333
Gerrit-PatchSet: 1
Gerrit-Owner: Michael Smith <[email protected]>
Gerrit-Reviewer: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Wed, 20 May 2026 19:53:28 +0000
Gerrit-HasComments: Yes

Reply via email to