David Knupp has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15796 )

Change subject: IMPALA-9649: Exclude shiro* and add to banned dependency maven 
plugin
......................................................................


Patch Set 6:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/15796/4/fe/pom.xml
File fe/pom.xml:

http://gerrit.cloudera.org:8080/#/c/15796/4/fe/pom.xml@1406
PS4, Line 1406:      Pin org.glassfish:javax.el explicitly.
              :
              :       HBase depends on this indirectly, and it's configured with
              :       a range of versions. This causes Maven to talk to all 
configured
              :       repositories, leading both to a lot of chattiness, and 
also
              :       failures if one of the repositories is unavailable.
              :       -->
              :       <dependency>
              :         <groupId>org.glassfish</groupId>
              :         <artifactId>javax.el</artifactId>
              :         <version>3.0.1-b08</version>
              :       </dependency>
              : 
              :       <!-- IMPALA-9
> I have sympathy for trying to find a way to get rid of these dependencies.
Done


http://gerrit.cloudera.org:8080/#/c/15796/4/fe/pom.xml@1422
PS4, Line 1422: hadoop-hdfs</artif
> This came from the approach I outlined above. I'm happy to try an alternate
Done



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9f9994bf81c1d2e025a03925e8eccb147c34d66e
Gerrit-Change-Number: 15796
Gerrit-PatchSet: 6
Gerrit-Owner: David Knupp <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Dinesh Garg (430)
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Wed, 29 Apr 2020 08:06:55 +0000
Gerrit-HasComments: Yes

Reply via email to