You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2021/10/13 12:08:26 UTC

[GitHub] [zeppelin] Reamer opened a new pull request #4254: [ZEPPELIN-5535] Remove jetty websocket libs

Reamer opened a new pull request #4254:
URL: https://github.com/apache/zeppelin/pull/4254


   ### What is this PR for?
   This PR removes more jetty websocket libs out of the code
   
   ### What type of PR is it?
    - Improvement
   
   ### What is the Jira issue?
   * https://issues.apache.org/jira/browse/ZEPPELIN-5535
   
   ### How should this be tested?
   * CI
   
   ### Questions:
   * Does the licenses files need update? No
   * Is there breaking changes for older versions? No
   * Does this needs documentation? No
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] jongyoul commented on pull request #4254: [ZEPPELIN-5535] Remove jetty websocket libs

Posted by GitBox <gi...@apache.org>.
jongyoul commented on pull request #4254:
URL: https://github.com/apache/zeppelin/pull/4254#issuecomment-942888223


   BTW, I thought more changes like deleting `NotebookSocket` itself. So I'll change more :-) 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] jongyoul closed pull request #4254: [ZEPPELIN-5535] Remove jetty websocket libs

Posted by GitBox <gi...@apache.org>.
jongyoul closed pull request #4254:
URL: https://github.com/apache/zeppelin/pull/4254


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] jongyoul commented on pull request #4254: [ZEPPELIN-5535] Remove jetty websocket libs

Posted by GitBox <gi...@apache.org>.
jongyoul commented on pull request #4254:
URL: https://github.com/apache/zeppelin/pull/4254#issuecomment-943008506


   I like this PR then I'll merge it and I'll make another PR with the same number of the issue :-) Thank you for taking care of it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [zeppelin] Reamer commented on pull request #4254: [ZEPPELIN-5535] Remove jetty websocket libs

Posted by GitBox <gi...@apache.org>.
Reamer commented on pull request #4254:
URL: https://github.com/apache/zeppelin/pull/4254#issuecomment-943006971


   > BTW, I thought more changes like deleting `NotebookSocket` itself. So I'll change more :-)
   
   I leave the pull request open, you can merge it and base your changes on it or close it and open a new pull request.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@zeppelin.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org