You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by za...@apache.org on 2022/05/17 09:35:51 UTC

[hive] branch master updated: HIVE-26226: Exclude jdk.tools dep from hive-metastore in upgrade-acid (Sylwester Lachiewicz, reviewed by Stamatis Zampetakis)

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

zabetak 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 d9724ad765e HIVE-26226: Exclude jdk.tools dep from hive-metastore in upgrade-acid (Sylwester Lachiewicz, reviewed by Stamatis Zampetakis)
d9724ad765e is described below

commit d9724ad765e38c2f29805a7d0a4660c5e467e3e7
Author: Sylwester Lachiewicz <sl...@apache.org>
AuthorDate: Thu May 12 11:58:00 2022 +0200

    HIVE-26226: Exclude jdk.tools dep from hive-metastore in upgrade-acid (Sylwester Lachiewicz, reviewed by Stamatis Zampetakis)
    
    The jdk.tools jars are not present in java versions > 8 thus there are
    build problems when compiling with newer JDKs.
    
    Exclude the dependency from hive-metastore (2.3.3) to avoid compilation
    problems in recent JDKs.
    
    It is safe to so because the dependency will still be fetched
    transitively by hadoop-common (2.7.2) when the appropriate maven (JDK)
    profile is in use.
    
    Closes #3284
---
 upgrade-acid/pre-upgrade/pom.xml | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/upgrade-acid/pre-upgrade/pom.xml b/upgrade-acid/pre-upgrade/pom.xml
index ea7044b5a69..9fbcee4de0a 100644
--- a/upgrade-acid/pre-upgrade/pom.xml
+++ b/upgrade-acid/pre-upgrade/pom.xml
@@ -71,6 +71,10 @@
           <groupId>org.apache.curator</groupId>
           <artifactId>curator-framework</artifactId>
         </exclusion>
+        <exclusion>
+          <groupId>jdk.tools</groupId>
+          <artifactId>jdk.tools</artifactId>
+        </exclusion>
       </exclusions>
     </dependency>
     <dependency>