You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2020/08/12 11:43:10 UTC

[GitHub] [calcite-avatica] petersomogyi opened a new pull request #125: [CALCITE-4095] Update Jetty to 9.4.31.v20200723

petersomogyi opened a new pull request #125:
URL: https://github.com/apache/calcite-avatica/pull/125


   Jetty upgrade also includes the migration to `SslContextFactory.Server` from `SslContextFactory`


----------------------------------------------------------------
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.

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

[GitHub] [calcite-avatica] petersomogyi commented on pull request #125: [CALCITE-4095] Update Jetty to 9.4.31.v20200723

Posted by GitBox <gi...@apache.org>.
petersomogyi commented on pull request #125:
URL: https://github.com/apache/calcite-avatica/pull/125#issuecomment-673422461


   I haven't seen any other incompatible changes in the new Jetty version.
   
   > Argh. Sorry I never merged this Peter. It must have gotten lost on my plate. Let me recheck this, given Kevin's feedback. (Or, I'm just remembering seeing this change elsewhere? :D)
   
   This was a fresh pull request. We had a similar change before at $dayjob covering the `SslContextFactory.Server` migration.
   
   Thanks for the reviews @risdenk and @joshelser!


----------------------------------------------------------------
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.

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

[GitHub] [calcite-avatica] joshelser commented on pull request #125: [CALCITE-4095] Update Jetty to 9.4.31.v20200723

Posted by GitBox <gi...@apache.org>.
joshelser commented on pull request #125:
URL: https://github.com/apache/calcite-avatica/pull/125#issuecomment-672986738


   Argh. Sorry I never merged this Peter. It must have gotten lost on my plate. Let me recheck this, given Kevin's feedback.


----------------------------------------------------------------
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.

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

[GitHub] [calcite-avatica] joshelser closed pull request #125: [CALCITE-4095] Update Jetty to 9.4.31.v20200723

Posted by GitBox <gi...@apache.org>.
joshelser closed pull request #125:
URL: https://github.com/apache/calcite-avatica/pull/125


   


----------------------------------------------------------------
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.

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

[GitHub] [calcite-avatica] joshelser edited a comment on pull request #125: [CALCITE-4095] Update Jetty to 9.4.31.v20200723

Posted by GitBox <gi...@apache.org>.
joshelser edited a comment on pull request #125:
URL: https://github.com/apache/calcite-avatica/pull/125#issuecomment-672986738


   Argh. Sorry I never merged this Peter. It must have gotten lost on my plate. Let me recheck this, given Kevin's feedback. (Or, I'm just remembering seeing this change elsewhere? :D)


----------------------------------------------------------------
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.

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