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

    https://github.com/apache/storm/pull/2752#discussion_r204259185
  
    --- Diff: storm-core/pom.xml ---
    @@ -32,6 +32,9 @@
         <properties>
             <worker-launcher.conf.dir>/etc/storm</worker-launcher.conf.dir>
             
<worker-launcher.build.dir>${project.build.directory}/native/worker-launcher</worker-launcher.build.dir>
    +        <jersey.version>2.27</jersey.version>
    +        <grizzly.version>2.4.3</grizzly.version>
    +        <jersey-grizzly2.version>1.19.4</jersey-grizzly2.version>
    --- End diff --
    
    I'd like to see this placed in web-app or individual module before merging 
this in. Once we pull this in and release, follow-up issues would be easy to 
forget. And our final goal for breaking down `storm-core` is to remove 
`storm-core`, so moving this to web-app also helps on reducing the work on this.


---

Reply via email to