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

    https://github.com/apache/storm/pull/2752#discussion_r200404317
  
    --- Diff: storm-core/pom.xml ---
    @@ -194,10 +249,7 @@
                 <artifactId>java-hamcrest</artifactId>
                 <scope>test</scope>
             </dependency>
    -        <dependency>
    -            <groupId>javax.servlet</groupId>
    -            <artifactId>javax.servlet-api</artifactId>
    -        </dependency>
    +        <!-- 
https://mvnrepository.com/artifact/javax.servlet/javax.servlet-api -->
    --- End diff --
    
    Not sure why we need the comment, if we removed the dependency.


---

Reply via email to