pan3793 commented on code in PR #4816:
URL: https://github.com/apache/zeppelin/pull/4816#discussion_r1767876757


##########
zeppelin-interpreter-parent/pom.xml:
##########
@@ -52,6 +53,10 @@
           <groupId>io.atomix</groupId>
           <artifactId>*</artifactId>
         </exclusion>
+        <exclusion>
+          <groupId>org.apache.hadoop</groupId>

Review Comment:
   @Reamer I do not mean the whole PR, just this
   
   ```patch
   + <exclusion>
   +   <groupId>org.apache.hadoop</groupId>
   +   <artifactId>*</artifactId>
   + </exclusion>
   ``` 
   
   ```
   ./mvnw clean package -Pbuild-distr -DskipTests
   ```
   
   without the above exclusion
   ```
   du -hd1 interpreter 
   7.5M    interpreter/neo4j
    15M    interpreter/sparql
    83M    interpreter/alluxio
   4.2M    interpreter/file
    31M    interpreter/python
    30M    interpreter/flink
   154M    interpreter/r
   476K    interpreter/angular
   772K    interpreter/java
   7.9M    interpreter/bigquery
   1.6M    interpreter/mongodb
   2.7M    interpreter/jdbc
   4.7M    interpreter/influxdb
    20M    interpreter/sh
    15M    interpreter/spark-submit
   7.5M    interpreter/livy
    39M    interpreter/cassandra
    22M    interpreter/md
    14M    interpreter/hbase
   7.2M    interpreter/groovy
    16M    interpreter/flink-cmd
    19M    interpreter/jupyter
    25M    interpreter/elasticsearch
    38M    interpreter/spark
   585M    interpreter
   ```
   
   with the above exclusion
   ```
   du -hd1 interpreter                           
   7.5M    interpreter/neo4j
    15M    interpreter/sparql
    96M    interpreter/alluxio
   4.2M    interpreter/file
    31M    interpreter/python
    30M    interpreter/flink
   166M    interpreter/r
   476K    interpreter/angular
   772K    interpreter/java
   7.9M    interpreter/bigquery
   1.6M    interpreter/mongodb
   2.7M    interpreter/jdbc
   4.7M    interpreter/influxdb
    20M    interpreter/sh
    15M    interpreter/spark-submit
   7.5M    interpreter/livy
    39M    interpreter/cassandra
    22M    interpreter/md
    14M    interpreter/hbase
   7.2M    interpreter/groovy
    16M    interpreter/flink-cmd
    19M    interpreter/jupyter
    26M    interpreter/elasticsearch
    38M    interpreter/spark
   612M    interpreter
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to