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

    https://github.com/apache/storm/pull/2738#discussion_r198541582
  
    --- Diff: storm-core/pom.xml ---
    @@ -170,6 +171,58 @@
                 <groupId>org.apache.logging.log4j</groupId>
                 <artifactId>log4j-slf4j-impl</artifactId>
             </dependency>
    +
    +        <dependency>
    +            <groupId>org.glassfish.jersey</groupId>
    +            <artifactId>jersey-bom</artifactId>
    +            <version>${jersey.version}</version>
    +            <type>pom</type>
    +            <scope>compile</scope>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.glassfish.jersey.containers</groupId>
    +            <artifactId>jersey-container-grizzly2-http</artifactId>
    +            <version>${jersey.version}</version>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.glassfish.jersey.inject</groupId>
    +            <artifactId>jersey-hk2</artifactId>
    +            <version>${jersey.version}</version>
    +        </dependency>
    +        <dependency>
    +            <groupId>org.glassfish.jersey.containers</groupId>
    +            <artifactId>jersey-container-grizzly2-servlet</artifactId>
    +            <version>${jersey.version}</version>
    +        </dependency>
    +        <dependency>
    +            <groupId>com.sun.jersey</groupId>
    +            <artifactId>jersey-grizzly2</artifactId>
    +            <version>1.19.4</version>
    +        </dependency>
    +        <dependency>
    +            <groupId>com.sun.jersey</groupId>
    +            <artifactId>jersey-grizzly2-servlet</artifactId>
    +            <version>1.19.4</version>
    +            <scope>test</scope>
    +        </dependency>
    +
    +        <dependency>
    +            <groupId>org.glassfish.grizzly</groupId>
    +            <artifactId>grizzly-framework</artifactId>
    +            <version>2.4.3</version>
    --- End diff --
    
    nit: and this one too, a property instead of a hard coded version, that is 
used in multiple places.


---

Reply via email to