[parquet-mr] branch master updated: PARQUET-1544: Possible over-shading of modules (#628)

[ga...@apache.org]

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

gabor pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/parquet-mr.git


The following commit(s) were added to refs/heads/master by this push:
     new 3e3049d  PARQUET-1544: Possible over-shading of modules (#628)
3e3049d is described below

commit 3e3049d042ffe033544e49ccac18a5edfc24adf0
Author: Gabor Szadovszky <ga...@apache.org>
AuthorDate: Tue Mar 19 13:58:01 2019 +0100

    PARQUET-1544: Possible over-shading of modules (#628)
---
 parquet-avro/pom.xml | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/parquet-avro/pom.xml b/parquet-avro/pom.xml
index 1649803..f422512 100644
--- a/parquet-avro/pom.xml
+++ b/parquet-avro/pom.xml
@@ -161,10 +161,31 @@
           </execution>
         </executions>
       </plugin>
+
+      <!-- Configuration to shade fastutil only instead of using the default configuration which shades jackson as well. -->
+      <!-- We shall not shade jackson here otherwise we may corrupt Avro API calls. (Avro API contains jackson classes.) -->
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-shade-plugin</artifactId>
+        <executions>
+          <execution>
+            <configuration combine.self="override">
+              <artifactSet>
+                <includes>
+                  <include>it.unimi.dsi:fastutil</include>
+                </includes>
+              </artifactSet>
+              <relocations>
+                <relocation>
+                  <pattern>it.unimi.dsi</pattern>
+                  <shadedPattern>${shade.prefix}.it.unimi.dsi</shadedPattern>
+                </relocation>
+              </relocations>
+            </configuration>
+          </execution>
+        </executions>
       </plugin>
+
     </plugins>
   </build>