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

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


The following commit(s) were added to refs/heads/master by this push:
     new bfaaad4  HIVE-25531: Remove the core classified hive-exec artifact 
(Zoltan Haindrich, reviewed by Stamatis Zampetakis)
bfaaad4 is described below

commit bfaaad4aa433963947d2a40020a432f19fc2b959
Author: Zoltan Haindrich <k...@rxd.hu>
AuthorDate: Thu Nov 18 16:17:58 2021 +0000

    HIVE-25531: Remove the core classified hive-exec artifact (Zoltan 
Haindrich, reviewed by Stamatis Zampetakis)
    
    Closes #2648
---
 itests/qtest-accumulo/pom.xml |  6 ------
 itests/qtest-kudu/pom.xml     |  6 ------
 ql/pom.xml                    | 21 ---------------------
 3 files changed, 33 deletions(-)

diff --git a/itests/qtest-accumulo/pom.xml b/itests/qtest-accumulo/pom.xml
index 7c03159..41c125f 100644
--- a/itests/qtest-accumulo/pom.xml
+++ b/itests/qtest-accumulo/pom.xml
@@ -61,12 +61,6 @@
       <groupId>org.apache.hive</groupId>
       <artifactId>hive-exec</artifactId>
       <scope>test</scope>
-      <classifier>core</classifier>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.hive</groupId>
-      <artifactId>hive-exec</artifactId>
-      <scope>test</scope>
       <classifier>tests</classifier>
     </dependency>
     <dependency>
diff --git a/itests/qtest-kudu/pom.xml b/itests/qtest-kudu/pom.xml
index 5a894bd..c83369b 100644
--- a/itests/qtest-kudu/pom.xml
+++ b/itests/qtest-kudu/pom.xml
@@ -50,12 +50,6 @@
       <groupId>org.apache.hive</groupId>
       <artifactId>hive-exec</artifactId>
       <scope>test</scope>
-      <classifier>core</classifier>
-    </dependency>
-    <dependency>
-      <groupId>org.apache.hive</groupId>
-      <artifactId>hive-exec</artifactId>
-      <scope>test</scope>
       <classifier>tests</classifier>
     </dependency>
     <dependency>
diff --git a/ql/pom.xml b/ql/pom.xml
index c82f965..4054bb3 100644
--- a/ql/pom.xml
+++ b/ql/pom.xml
@@ -1037,26 +1037,6 @@
             </goals>
           </execution>
           <execution>
-            <!--this 'execution' will produce a hive-exec-$version-core.jar 
which has
-            all classes from ql module, i.e. the same contents as 
hive-exec-$version.jar would
-            have had if there was shade plugin below.  The way shade was 
configured since 0.13, is
-            to override the default jar for ql module with the saded one but 
keep the same name.
-            In other words, ql modules producesd a single uber jar under the 
module's coordinates,
-            i.e. w/o a classifier.  We also need a slim jar that only has 
classes from ql.  This
-            jar will now be with a 'core' classifier to ensure that prior 
behavior is preserved.
-            It would have been better to have a slim jar called 
hive-exec-$version.jar and a fat
-            jar called hive-exec-$version-shaded.jar but doing this now will 
cause backwards compat.
-            issues-->
-            <id>core-jar</id>
-            <phase>package</phase>
-            <goals>
-              <goal>jar</goal>
-            </goals>
-            <configuration>
-              <classifier>core</classifier>
-            </configuration>
-          </execution>
-          <execution>
             <id>fallbackauthorizer-jar</id>
             <phase>package</phase>
             <goals>
@@ -1082,7 +1062,6 @@
               <goal>shade</goal>
             </goals>
             <configuration>
-                <!--also see maven-jar-plugin execution.id=core-jar-->
               <artifactSet>
                 <includes>
                   <!-- order is meant to be the same as the ant build -->

Reply via email to