David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/15829 )
Change subject: IMPALA-9648: Exclude/ban netty* from mvn download ...................................................................... Patch Set 6: (3 comments) http://gerrit.cloudera.org:8080/#/c/15829/4/fe/pom.xml File fe/pom.xml: http://gerrit.cloudera.org:8080/#/c/15829/4/fe/pom.xml@104 PS4, Line 104: <dependency> : <groupId>org.apache.hadoop</groupId> : <artifactId>hadoop-auth</artifactId> : <version>${hadoop.version}</version> : <exclusions> : <!-- https://issues.apache.org/jira/browse/HADOOP-14903 --> : <exclusion> : <groupId>net.minidev</groupId> : <artifactId>json-smart</artifactId> : </exclusion> : </exclusions> : </dependency> > Maybe this is leftover from something else, but we don't want to add any de Yeah, I think a leftover from my earlier attempt where I had the profile confusion. Done. http://gerrit.cloudera.org:8080/#/c/15829/4/fe/pom.xml@783 PS4, Line 783: ommon dep > Something that I forgot to do in my version is we should just exclude every Done http://gerrit.cloudera.org:8080/#/c/15829/4/fe/pom.xml@1421 PS4, Line 1421: > Nit: remove this stray line. Done -- To view, visit http://gerrit.cloudera.org:8080/15829 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ie7d61af3c10ee439ca9eef3840403229e6235c97 Gerrit-Change-Number: 15829 Gerrit-PatchSet: 6 Gerrit-Owner: David Knupp <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Comment-Date: Sat, 02 May 2020 00:42:53 +0000 Gerrit-HasComments: Yes
