Csaba Ringhofer 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: Code-Review+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 sure why: https://github.com/apache/impala/commit/f2a602ea9d7774a13c5866ad294e2aaee677ae62 Maybe we wanted to depend on a not yet released cdh version. -- 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-Comment-Date: Mon, 05 Oct 2020 21:40:40 +0000 Gerrit-HasComments: Yes
