Peter Rozsa has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24598 )

Change subject: IMPALA-9153: Generate CUP parser Java source via CMake
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/24598/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/24598/2//COMMIT_MSG@17
PS2, Line 17: maintaining our own custom Maven plugin for our version
> The basic situation is that we do need our specific version of CUP and ther
Ack


http://gerrit.cloudera.org:8080/#/c/24598/2/bin/cmake_aux/generate-cup-sources.sh
File bin/cmake_aux/generate-cup-sources.sh:

http://gerrit.cloudera.org:8080/#/c/24598/2/bin/cmake_aux/generate-cup-sources.sh@28
PS2, Line 28: JAVA_CUP_JAR=java-cup-0.11-a-czt02-cdh.jar
> We do use it in multiple places, I'll take a look
Ack



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I180b191b72b54e3661726e1560be844341ed92e5
Gerrit-Change-Number: 24598
Gerrit-PatchSet: 2
Gerrit-Owner: Joe McDonnell <[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-Reviewer: Peter Rozsa <[email protected]>
Gerrit-Comment-Date: Tue, 21 Jul 2026 07:14:21 +0000
Gerrit-HasComments: Yes

Reply via email to