You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@storm.apache.org by Ethanlm <gi...@git.apache.org> on 2017/10/25 05:02:17 UTC

[GitHub] storm pull request #2386: [STORM-1927] upgrade to jetty 9

GitHub user Ethanlm opened a pull request:

    https://github.com/apache/storm/pull/2386

    [STORM-1927] upgrade to jetty 9

    We need to upgrade to jetty latest version. The current jetty version is really old

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/Ethanlm/storm STORM-1927

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/2386.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2386
    
----
commit 7a61f732d4d369083def41529cf027772b87bf5f
Author: Ethan Li <et...@gmail.com>
Date:   2017-10-24T22:12:56Z

    [STORM-1927] upgrade to jetty 9

----


---

[GitHub] storm pull request #2386: [STORM-1927] upgrade to jetty 9

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/storm/pull/2386


---

[GitHub] storm issue #2386: [STORM-1927] upgrade to jetty 9

Posted by Ethanlm <gi...@git.apache.org>.

Github user Ethanlm commented on the issue:

    https://github.com/apache/storm/pull/2386
  
    Thanks @revans2  for pointing it out. I made stupid mistakes.  I am still fixing it.  Please don't merge yet.


---

[GitHub] storm issue #2386: [STORM-1927] upgrade to jetty 9

Posted by Ethanlm <gi...@git.apache.org>.

Github user Ethanlm commented on the issue:

    https://github.com/apache/storm/pull/2386
  
    I don't know why the storm-core is waiting for so long and not yet started. 
    https://travis-ci.org/apache/storm/builds/292621349
    
     However, the build on my repo https://travis-ci.org/Ethanlm/storm/builds/292621326 passed.


---

[GitHub] storm issue #2386: [STORM-1927] upgrade to jetty 9

Posted by revans2 <gi...@git.apache.org>.

Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2386
  
    I am getting the UI crashing on startup, but with no stacktrace in the logs.  Could you look into it?


---

[GitHub] storm issue #2386: [STORM-1927] upgrade to jetty 9

Posted by Ethanlm <gi...@git.apache.org>.

Github user Ethanlm commented on the issue:

    https://github.com/apache/storm/pull/2386
  
    I fixed the bugs and tested it. It should be good now. 


---