gabrywu commented on code in PR #2588:
URL: https://github.com/apache/incubator-kyuubi/pull/2588#discussion_r867354133


##########
externals/kyuubi-hive-sql-engine/pom.xml:
##########
@@ -159,63 +159,63 @@
         
<outputDirectory>target/scala-${scala.binary.version}/classes</outputDirectory>
         
<testOutputDirectory>target/scala-${scala.binary.version}/test-classes</testOutputDirectory>
         <plugins>
-        <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-shade-plugin</artifactId>
-            <configuration>
-                <shadedArtifactAttached>false</shadedArtifactAttached>
-                <artifactSet>
-                    <includes>
-                        <include>org.scala-lang:scala-library</include>
-                        
<include>com.fasterxml.jackson.core:jackson-annotations</include>
-                        
<include>com.fasterxml.jackson.core:jackson-core</include>
-                        
<include>com.fasterxml.jackson.core:jackson-databind</include>
-                        
<include>com.fasterxml.jackson.module:jackson-module-scala_2.12</include>
-                        
<include>org.apache.kyuubi:kyuubi-common_${scala.binary.version}</include>
-                        
<include>org.apache.kyuubi:kyuubi-events_${scala.binary.version}</include>
-                        
<include>org.apache.kyuubi:kyuubi-ha_${scala.binary.version}</include>
-                        <include>org.apache.curator:curator-client</include>
-                        <include>org.apache.curator:curator-framework</include>
-                        <include>org.apache.curator:curator-recipes</include>
-                    </includes>
-                </artifactSet>
-                <filters>
-                    <filter>
-                        <artifact>*:*</artifact>
-                        <excludes>
-                            <exclude>META-INF/*.SF</exclude>
-                            <exclude>META-INF/*.DSA</exclude>
-                            <exclude>META-INF/*.RSA</exclude>
-                            <exclude>META-INF/DEPENDENCIES</exclude>
-                            <exclude>META-INF/LICENSE.txt</exclude>
-                            <exclude>META-INF/NOTICE.txt</exclude>
-                            <exclude>META-INF/maven/**</exclude>
-                            <exclude>LICENSE.txt</exclude>
-                            <exclude>NOTICE.txt</exclude>
-                            <exclude>mozilla/**</exclude>
-                            <exclude>**/module-info.class</exclude>
-                        </excludes>
-                    </filter>
-                </filters>
-                <relocations>
-                    <relocation>
-                        <pattern>org.apache.curator</pattern>
-                        
<shadedPattern>${kyuubi.shade.packageName}.org.apache.curator</shadedPattern>
+            <plugin>
+                <groupId>org.apache.maven.plugins</groupId>
+                <artifactId>maven-shade-plugin</artifactId>
+                <configuration>
+                    <shadedArtifactAttached>false</shadedArtifactAttached>
+                    <artifactSet>
                         <includes>
-                            <include>org.apache.curator.**</include>
+                            <include>org.scala-lang:scala-library</include>
+                            
<include>com.fasterxml.jackson.core:jackson-annotations</include>
+                            
<include>com.fasterxml.jackson.core:jackson-core</include>
+                            
<include>com.fasterxml.jackson.core:jackson-databind</include>
+                            
<include>com.fasterxml.jackson.module:jackson-module-scala_2.12</include>

Review Comment:
   done



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to