[ 
https://issues.apache.org/jira/browse/STORM-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15176890#comment-15176890
 ] 

ASF GitHub Bot commented on STORM-822:
--------------------------------------

Github user hmcl commented on a diff in the pull request:

    https://github.com/apache/storm/pull/1131#discussion_r54822068
  
    --- Diff: pom.xml ---
    @@ -836,14 +839,39 @@
                     <scope>test</scope>
                 </dependency>
                 <dependency>
    -              <groupId>org.apache.calcite</groupId>
    -              <artifactId>calcite-core</artifactId>
    -              <version>${calcite.version}</version>
    +                <groupId>org.apache.calcite</groupId>
    +                <artifactId>calcite-core</artifactId>
    +                <version>${calcite.version}</version>
    +            </dependency>
    +            <dependency>
    +                <groupId>com.fasterxml.jackson.core</groupId>
    +                <artifactId>jackson-databind</artifactId>
    +                <version>${jackson.version}</version>
    +            </dependency>
    +            <dependency>
    +                <groupId>org.apache.kafka</groupId>
    +                <artifactId>${kafka.artifact.id}</artifactId>
    +                <version>${kafka.version}</version>
    +                <exclusions>
    +                    <exclusion>
    +                        <groupId>org.apache.zookeeper</groupId>
    +                        <artifactId>zookeeper</artifactId>
    +                    </exclusion>
    +                    <exclusion>
    +                        <groupId>log4j</groupId>
    +                        <artifactId>log4j</artifactId>
    +                    </exclusion>
    +                    <exclusion>
    +                        <groupId>org.slf4j</groupId>
    +                        <artifactId>slf4j-log4j12</artifactId>
    +                    </exclusion>
    +                </exclusions>
                 </dependency>
                 <dependency>
    -              <groupId>com.fasterxml.jackson.core</groupId>
    -              <artifactId>jackson-databind</artifactId>
    -              <version>${jackson.version}</version>
    +                <groupId>org.apache.kafka</groupId>
    +                <artifactId>kafka-clients</artifactId>
    +                <version>${kafka.version}</version>
    +                <scope>provided</scope>
    --- End diff --
    
    @revans2 the following pom files already had provided before and I left 
them in the same state. Should we leave it as provided, or change that to the 
default (compile) ?
    
    storm-starter/pom.xml
    flux-examples/pom.xml
    storm-sql-kafka/pom.xml


> As a storm developer I’d like to use the new kafka consumer API (0.8.3) to 
> reduce dependencies and use long term supported kafka apis 
> --------------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: STORM-822
>                 URL: https://issues.apache.org/jira/browse/STORM-822
>             Project: Apache Storm
>          Issue Type: Story
>          Components: storm-kafka
>            Reporter: Thomas Becker
>            Assignee: Hugo Louro
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to