This is an automated email from the ASF dual-hosted git repository.

snlee pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new c1ac8a1  Do not shade jar for pinot native plug-in components (#7245)
c1ac8a1 is described below

commit c1ac8a18b65841fc666722496cc5f4f9347b3dd4
Author: Seunghyun Lee <sn...@linkedin.com>
AuthorDate: Tue Aug 3 11:39:45 2021 -0700

    Do not shade jar for pinot native plug-in components (#7245)
    
    We create the shaded jars for plug-in modules by default; however,
    this results in a side effect of large binary release data size.
    This pr don't create shaded jars for pinot native plug-in
    components. This reduces the binary release from `848MB` to `566MB`.
---
 pinot-distribution/pinot-assembly.xml                        | 12 ++++++------
 .../pinot-minion-tasks/pinot-minion-builtin-tasks/pom.xml    |  2 +-
 .../pinot-segment-uploader-default/pom.xml                   |  2 +-
 .../pinot-segment-writer-file-based/pom.xml                  |  2 +-
 4 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/pinot-distribution/pinot-assembly.xml 
b/pinot-distribution/pinot-assembly.xml
index 6da83f3..29e42f9 100644
--- a/pinot-distribution/pinot-assembly.xml
+++ b/pinot-distribution/pinot-assembly.xml
@@ -134,8 +134,8 @@
     <!-- End Include Pinot Input Format Plugins-->
     <!-- Start Include Pinot Minion Tasks Plugins-->
     <file>
-      
<source>${pinot.root}/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/target/pinot-minion-builtin-tasks-${project.version}-shaded.jar</source>
-      
<destName>plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pinot-minion-builtin-tasks-${project.version}-shaded.jar</destName>
+      
<source>${pinot.root}/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/target/pinot-minion-builtin-tasks-${project.version}.jar</source>
+      
<destName>plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pinot-minion-builtin-tasks-${project.version}.jar</destName>
     </file>
     <!-- End Include Pinot Minion Tasks Plugins-->
     <!-- Start Include Pinot Metrics Plugins-->
@@ -146,14 +146,14 @@
     <!-- End Include Pinot Metrics Plugins-->
     <!-- Start Include Pinot Segment Writer Plugins-->
     <file>
-      
<source>${pinot.root}/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/target/pinot-segment-writer-file-based-${project.version}-shaded.jar</source>
-      
<destName>plugins/pinot-segment-writer/pinot-segment-writer-file-based/pinot-segment-writer-file-based-${project.version}-shaded.jar</destName>
+      
<source>${pinot.root}/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/target/pinot-segment-writer-file-based-${project.version}.jar</source>
+      
<destName>plugins/pinot-segment-writer/pinot-segment-writer-file-based/pinot-segment-writer-file-based-${project.version}.jar</destName>
     </file>
     <!-- End Include Pinot Segment Writer Plugins-->
     <!-- Start Include Pinot Segment Uploader Plugins-->
     <file>
-      
<source>${pinot.root}/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/target/pinot-segment-uploader-default-${project.version}-shaded.jar</source>
-      
<destName>plugins/pinot-segment-uploader/pinot-segment-uploader-default/pinot-segment-uploader-default-${project.version}-shaded.jar</destName>
+      
<source>${pinot.root}/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/target/pinot-segment-uploader-default-${project.version}.jar</source>
+      
<destName>plugins/pinot-segment-uploader/pinot-segment-uploader-default/pinot-segment-uploader-default-${project.version}.jar</destName>
     </file>
     <!-- End Include Pinot Segment Uploader Plugins-->
     <!-- End Include Pinot Plugins-->
diff --git 
a/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pom.xml 
b/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pom.xml
index 80146d7..15fcd22 100644
--- a/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pom.xml
+++ b/pinot-plugins/pinot-minion-tasks/pinot-minion-builtin-tasks/pom.xml
@@ -34,7 +34,7 @@
   <url>https://pinot.apache.org/</url>
   <properties>
     <pinot.root>${basedir}/../../..</pinot.root>
-    <phase.prop>package</phase.prop>
+    <phase.prop>none</phase.prop>
   </properties>
 
   <build>
diff --git 
a/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/pom.xml 
b/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/pom.xml
index ae7e572..99b1b88 100644
--- 
a/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/pom.xml
+++ 
b/pinot-plugins/pinot-segment-uploader/pinot-segment-uploader-default/pom.xml
@@ -35,7 +35,7 @@
   <url>https://pinot.apache.org/</url>
   <properties>
     <pinot.root>${basedir}/../../..</pinot.root>
-    <phase.prop>package</phase.prop>
+    <phase.prop>none</phase.prop>
   </properties>
 
   <dependencies>
diff --git 
a/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/pom.xml 
b/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/pom.xml
index 5dcb164..d886c1a 100644
--- a/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/pom.xml
+++ b/pinot-plugins/pinot-segment-writer/pinot-segment-writer-file-based/pom.xml
@@ -35,7 +35,7 @@
   <url>https://pinot.apache.org/</url>
   <properties>
     <pinot.root>${basedir}/../../..</pinot.root>
-    <phase.prop>package</phase.prop>
+    <phase.prop>none</phase.prop>
   </properties>
 
   <dependencies>

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to