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

Change subject: IMPALA-9648: Exclude/ban netty-all from mvn download
......................................................................


Patch Set 4:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/15829/3/fe/pom.xml
File fe/pom.xml:

http://gerrit.cloudera.org:8080/#/c/15829/3/fe/pom.xml@762
PS3, Line 762:
> TL;DR
Done


http://gerrit.cloudera.org:8080/#/c/15829/3/fe/pom.xml@1386
PS3, Line 1386:           <name>m2e.version</name>
              :         </property>
              :       </activation>
              :         <!-- By default, we separate Eclipse-built files from 
Maven-built
              :              files. Otherwise, they are both in target/classes, 
and Eclipse
              :              and Maven may clobber each other, complicating 
attaching to
              :              a running process. -->
              :         <properties>
              :           
<buildOutputDirectory>${project.build.directory}/${eclipse.output.directory}</buildOutputDirectory>
              :         </properties>
              :     </profile>
              :
              :     <profile>
              :       <id>s3-tests</id>
              :       <properties>
              :         
<testcase.groups>org.apache.impala.planner.S3Tests</testcase.groups>
              :       </properties>
              :     </profile>
              :   </profiles>
              :
              :   <dependencyManagement>
              :     <dependencies>
              :       <!--
              :       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
> The check job posted a comment on this gerrit review. It did a build and th
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: 4
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: Fri, 01 May 2020 18:46:20 +0000
Gerrit-HasComments: Yes

Reply via email to