Xerial Snappy has a JNI component so cannot be shaded or else native …

…symbols won't link
apache · Jan 13, 2025 · 2c69208 · 2c69208
1 parent 858ea51
commit 2c69208
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/phoenix-client-parent/pom.xml b/phoenix-client-parent/pom.xml
@@ -165,6 +165,9 @@
                   <!-- Phoenix classes -->
                   <exclude>org/apache/phoenix/**</exclude>
                   <exclude>org/apache/omid/**</exclude>
+                  <!-- Xerial Snappy has a JNI component so cannot be shaded or
+                       else native symbols won't link -->
+                  <exclude>org/xerial/snappy/**</exclude>
                 </excludes>
               </relocation>
               <relocation>

diff --git a/phoenix-mapreduce-byo-shaded-hbase/pom.xml b/phoenix-mapreduce-byo-shaded-hbase/pom.xml
@@ -175,7 +175,10 @@
                   <!-- See PHOENIX-7118
                  We expect this to come from the unshaded hadoop on the classpath.
                  This won't work if hadoop-shaded-client is used. -->
-                <exclude>org/apache/commons/configuration2/**</exclude>
+                  <exclude>org/apache/commons/configuration2/**</exclude>
+                  <!-- Xerial Snappy has a JNI component so cannot be shaded or
+                       else native symbols won't link -->
+                  <exclude>org/xerial/snappy/**</exclude>
                 </excludes>
               </relocation>
 

diff --git a/phoenix-server/pom.xml b/phoenix-server/pom.xml
@@ -178,7 +178,10 @@
                   <!-- See PHOENIX-7118
                  We expect this to come from the unshaded hadoop on the classpath.
                  That is always true for the HBase server classpath -->
-                <exclude>org/apache/commons/configuration2/**</exclude>
+                  <exclude>org/apache/commons/configuration2/**</exclude>
+                  <!-- Xerial Snappy has a JNI component so cannot be shaded or
+                       else native symbols won't link -->
+                  <exclude>org/xerial/snappy/**</exclude>
                 </excludes>
               </relocation>
               <!-- We use the relocate-by default settings from -client, instead of relocating