[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-09-01 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909055195 -- 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 unsub

[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-08-31 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909055195 -- 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 unsub

[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-08-31 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909079950 This is the problem This Jetty dependency `http2-http-client-transport ` adds transitive dependencies depending on the JDK version. This is very bad because the build is

[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-08-31 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909073901 it looks like it is the HTTP Airlift dependency in Presto SQL distribution with jdk8: ``` mvn dependency:tree [INFO] | +- io.airlift:http-client:jar:0.195:comp

[GitHub] [pulsar] eolivelli commented on pull request #11859: [Branch-2.8]Fix jetty-alpn-java-client license

2021-08-31 Thread GitBox
eolivelli commented on pull request #11859: URL: https://github.com/apache/pulsar/pull/11859#issuecomment-909055195 I want to investigate why the result is different. those should be Maven artifact ids and they should not depend on the jdk -- This is an automated message from the Apac