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/16 15:23:02 UTC

[hive] branch master updated: HIVE-26205: Incorrect scope for slf4j-api dependency in kafka-handler (Wechar Yu, 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 74e3f29b432 HIVE-26205: Incorrect scope for slf4j-api dependency in kafka-handler (Wechar Yu, reviewed by Stamatis Zampetakis)
74e3f29b432 is described below

commit 74e3f29b4328b5df5a1cd90de12ec3e35301afae
Author: wecharyu <we...@shopee.com>
AuthorDate: Sat May 7 20:56:54 2022 +0800

    HIVE-26205: Incorrect scope for slf4j-api dependency in kafka-handler (Wechar Yu, reviewed by Stamatis Zampetakis)
    
    The classes in the kafka-handler module are using the slf4j-api thus
    this dependency must be set at compile scope. Currently it is set at
    test scope which makes the build fail in some recent maven versions (
    e.g., 3.8.5).
    
    The parent pom declares explicitly slf4j-api at compile scope so
    removing all the references from kafka-handler/pom.xml is the way to
    go.
    
    Closes #3272
---
 kafka-handler/pom.xml | 21 ---------------------
 1 file changed, 21 deletions(-)

diff --git a/kafka-handler/pom.xml b/kafka-handler/pom.xml
index 4f4f8fab040..a9156848f32 100644
--- a/kafka-handler/pom.xml
+++ b/kafka-handler/pom.xml
@@ -43,12 +43,6 @@
       <artifactId>hive-exec</artifactId>
       <scope>provided</scope>
       <version>${project.version}</version>
-      <exclusions>
-        <exclusion>
-          <groupId>org.slf4j</groupId>
-          <artifactId>slf4j-api</artifactId>
-        </exclusion>
-      </exclusions>
     </dependency>
     <dependency>
       <groupId>org.apache.hive</groupId>
@@ -67,10 +61,6 @@
           <groupId>commons-beanutils</groupId>
           <artifactId>commons-beanutils</artifactId>
         </exclusion>
-        <exclusion>
-          <groupId>org.slf4j</groupId>
-          <artifactId>slf4j-api</artifactId>
-        </exclusion>
         <exclusion>
           <groupId>org.yaml</groupId>
           <artifactId>snakeyaml</artifactId>
@@ -80,12 +70,6 @@
     <dependency>
       <groupId>org.apache.hadoop</groupId>
       <artifactId>hadoop-client</artifactId>
-      <exclusions>
-        <exclusion>
-          <groupId>org.slf4j</groupId>
-          <artifactId>slf4j-api</artifactId>
-        </exclusion>
-      </exclusions>
     </dependency>
     <dependency>
       <groupId>org.apache.kafka</groupId>
@@ -127,11 +111,6 @@
       <artifactId>zookeeper</artifactId>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>org.slf4j</groupId>
-      <artifactId>slf4j-api</artifactId>
-      <scope>test</scope>
-    </dependency>
     <dependency>
       <groupId>io.confluent</groupId>
       <artifactId>kafka-avro-serializer</artifactId>