Joe McDonnell has posted comments on this change. ( http://gerrit.cloudera.org:8080/16543 )
Change subject: IMPALA-10218: Remove impala.cdh.repo Maven repository ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/16543/2/impala-parent/pom.xml File impala-parent/pom.xml: http://gerrit.cloudera.org:8080/#/c/16543/2/impala-parent/pom.xml@149 PS2, Line 149: <repository> : <id>cdh.rcs.releases.repo</id> : <url>https://repository.cloudera.com/content/groups/cdh-releases-rcs</url> : <name>CDH Releases Repository</name> : <snapshots> : <enabled>true</enabled> : </snapshots> : </repository> > Do you know if we still need this? Was introduced a long time ago, I am not We do get some artifacts from that repo. Here is the list based on parsing the build-all-options maven output: org.cloudera.logredactor: logredactor-2.0.8.jar org.codehaus.jackson.jackson-mapper-asl: jackson-mapper-asl-1.9.13-cloudera.1.jar log4j: log4j-1.2.17-cloudera1.jar org.codehaus.jackson.jackson-core-asl: jackson-core-asl-1.9.13-cloudera.1.jar net.sourceforge.czt.dev.java-cup-runtime: java-cup-runtime-0.11-a-czt01-cdh.jar net.sourceforce.cztv.dev.java-cup: java-cup-0.11-a-czt02-cdh.jar So, it looks like these are Cloudera-specific versions of things. My guess is that these apply patches for some security vulnerabilities on top of the versions used by Hadoop, etc. They must not get included in the impala.cdp.repo. I think we can disable snapshots for this repo. I will verify that works and upload a new patch. -- To view, visit http://gerrit.cloudera.org:8080/16543 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I44b587f936ae20c207c74a9800cf98baa464164a Gerrit-Change-Number: 16543 Gerrit-PatchSet: 2 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Comment-Date: Mon, 05 Oct 2020 22:01:14 +0000 Gerrit-HasComments: Yes
