You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pulsar.apache.org by tison <wa...@gmail.com> on 2022/06/22 15:36:42 UTC
Required and recommended JavaVersion for Pulsar broker
Hi devs.
During a review of PR-16163[1] I notice a comment from @Enrico Olivelli
<eo...@gmail.com> on PR-14953 said:
> We are currently still supporting jdk8 (on the server side)
However, on our README.md it recommends:
Pulsar Runtime Java Version Recommendation
pulsar master branch
Broker JavaVersion: 17
Also from the pom.xml file it shows:
<maven.compiler.source>17</maven.compiler.source>
<maven.compiler.target>17</maven.compiler.target>
<pulsar.broker.compiler.release>${maven.compiler.target}</pulsar.broker.compiler.release>
<pulsar.client.compiler.release>8</pulsar.client.compiler.release>
I'm unsure what JavaVersion required and recommend for Pulsar broker.
IIUC https://github.com/apache/pulsar/pull/15264 and PIP-156 (
https://github.com/apache/pulsar/issues/15207) are ongoing efforts to run
pulsar broker with JDK17.
Best,
tison.
[1] https://github.com/apache/pulsar/pull/16163
Re: Required and recommended JavaVersion for Pulsar broker
Posted by tison <wa...@gmail.com>.
Thanks for your information!
Then I think we can merge PR-16163 to master and release with 2.11, while
not to cherry-pick to <=2.10.
Best,
tison.
Ran Gao <rg...@apache.org> 于2022年6月23日周四 10:01写道：
> One problem is that the presto 334 needs jdk11+ and it didn't support
> jdk8, maybe 2.11 is a good time for Pulsar to upgrade the Presto to 334+ or
> the latest stable version.
>
> On 2022/06/22 18:22:09 Enrico Olivelli wrote:
> > For Pulsar 2.9 and 2.10 you should use jdk11
> > you can checkout the official docker images
> >
> > for 2.11 we decided to go with jdk17
> >
> > Enrico
> >
> > Il giorno mer 22 giu 2022 alle ore 17:42 ZhangJian He
> > <sh...@gmail.com> ha scritto:
> > >
> > > Enrico commented on 31 Mar. Now it's required jdk17 on master branch.
> > > `branch-2.10` is compatible with jdk8.
> > >
> > > Thanks
> > > ZhangJian He
> > >
> > > tison <wa...@gmail.com> 于2022年6月22日周三 23:37写道：
> > >
> > > > Hi devs.
> > > >
> > > > During a review of PR-16163[1] I notice a comment from @Enrico
> Olivelli
> > > > <eo...@gmail.com> on PR-14953 said:
> > > >
> > > > > We are currently still supporting jdk8 (on the server side)
> > > >
> > > > However, on our README.md it recommends:
> > > >
> > > > Pulsar Runtime Java Version Recommendation
> > > > pulsar master branch
> > > > Broker JavaVersion: 17
> > > >
> > > > Also from the pom.xml file it shows:
> > > >
> > > > <maven.compiler.source>17</maven.compiler.source>
> > > > <maven.compiler.target>17</maven.compiler.target>
> > > >
> > > >
> > > >
> <pulsar.broker.compiler.release>${maven.compiler.target}</pulsar.broker.compiler.release>
> > > >
> <pulsar.client.compiler.release>8</pulsar.client.compiler.release>
> > > >
> > > > I'm unsure what JavaVersion required and recommend for Pulsar broker.
> > > >
> > > > IIUC https://github.com/apache/pulsar/pull/15264 and PIP-156 (
> > > > https://github.com/apache/pulsar/issues/15207) are ongoing efforts
> to run
> > > > pulsar broker with JDK17.
> > > >
> > > > Best,
> > > > tison.
> > > >
> > > > [1] https://github.com/apache/pulsar/pull/16163
> > > >
> >
>
Re: Required and recommended JavaVersion for Pulsar broker
Posted by Ran Gao <rg...@apache.org>.
One problem is that the presto 334 needs jdk11+ and it didn't support jdk8, maybe 2.11 is a good time for Pulsar to upgrade the Presto to 334+ or the latest stable version.
On 2022/06/22 18:22:09 Enrico Olivelli wrote:
> For Pulsar 2.9 and 2.10 you should use jdk11
> you can checkout the official docker images
>
> for 2.11 we decided to go with jdk17
>
> Enrico
>
> Il giorno mer 22 giu 2022 alle ore 17:42 ZhangJian He
> <sh...@gmail.com> ha scritto:
> >
> > Enrico commented on 31 Mar. Now it's required jdk17 on master branch.
> > `branch-2.10` is compatible with jdk8.
> >
> > Thanks
> > ZhangJian He
> >
> > tison <wa...@gmail.com> 于2022年6月22日周三 23:37写道：
> >
> > > Hi devs.
> > >
> > > During a review of PR-16163[1] I notice a comment from @Enrico Olivelli
> > > <eo...@gmail.com> on PR-14953 said:
> > >
> > > > We are currently still supporting jdk8 (on the server side)
> > >
> > > However, on our README.md it recommends:
> > >
> > > Pulsar Runtime Java Version Recommendation
> > > pulsar master branch
> > > Broker JavaVersion: 17
> > >
> > > Also from the pom.xml file it shows:
> > >
> > > <maven.compiler.source>17</maven.compiler.source>
> > > <maven.compiler.target>17</maven.compiler.target>
> > >
> > >
> > > <pulsar.broker.compiler.release>${maven.compiler.target}</pulsar.broker.compiler.release>
> > > <pulsar.client.compiler.release>8</pulsar.client.compiler.release>
> > >
> > > I'm unsure what JavaVersion required and recommend for Pulsar broker.
> > >
> > > IIUC https://github.com/apache/pulsar/pull/15264 and PIP-156 (
> > > https://github.com/apache/pulsar/issues/15207) are ongoing efforts to run
> > > pulsar broker with JDK17.
> > >
> > > Best,
> > > tison.
> > >
> > > [1] https://github.com/apache/pulsar/pull/16163
> > >
>
Re: Required and recommended JavaVersion for Pulsar broker
Posted by Enrico Olivelli <eo...@gmail.com>.
For Pulsar 2.9 and 2.10 you should use jdk11
you can checkout the official docker images
for 2.11 we decided to go with jdk17
Enrico
Il giorno mer 22 giu 2022 alle ore 17:42 ZhangJian He
<sh...@gmail.com> ha scritto:
>
> Enrico commented on 31 Mar. Now it's required jdk17 on master branch.
> `branch-2.10` is compatible with jdk8.
>
> Thanks
> ZhangJian He
>
> tison <wa...@gmail.com> 于2022年6月22日周三 23:37写道：
>
> > Hi devs.
> >
> > During a review of PR-16163[1] I notice a comment from @Enrico Olivelli
> > <eo...@gmail.com> on PR-14953 said:
> >
> > > We are currently still supporting jdk8 (on the server side)
> >
> > However, on our README.md it recommends:
> >
> > Pulsar Runtime Java Version Recommendation
> > pulsar master branch
> > Broker JavaVersion: 17
> >
> > Also from the pom.xml file it shows:
> >
> > <maven.compiler.source>17</maven.compiler.source>
> > <maven.compiler.target>17</maven.compiler.target>
> >
> >
> > <pulsar.broker.compiler.release>${maven.compiler.target}</pulsar.broker.compiler.release>
> > <pulsar.client.compiler.release>8</pulsar.client.compiler.release>
> >
> > I'm unsure what JavaVersion required and recommend for Pulsar broker.
> >
> > IIUC https://github.com/apache/pulsar/pull/15264 and PIP-156 (
> > https://github.com/apache/pulsar/issues/15207) are ongoing efforts to run
> > pulsar broker with JDK17.
> >
> > Best,
> > tison.
> >
> > [1] https://github.com/apache/pulsar/pull/16163
> >
Re: Required and recommended JavaVersion for Pulsar broker
Posted by ZhangJian He <sh...@gmail.com>.
Enrico commented on 31 Mar. Now it's required jdk17 on master branch.
`branch-2.10` is compatible with jdk8.
Thanks
ZhangJian He
tison <wa...@gmail.com> 于2022年6月22日周三 23:37写道：
> Hi devs.
>
> During a review of PR-16163[1] I notice a comment from @Enrico Olivelli
> <eo...@gmail.com> on PR-14953 said:
>
> > We are currently still supporting jdk8 (on the server side)
>
> However, on our README.md it recommends:
>
> Pulsar Runtime Java Version Recommendation
> pulsar master branch
> Broker JavaVersion: 17
>
> Also from the pom.xml file it shows:
>
> <maven.compiler.source>17</maven.compiler.source>
> <maven.compiler.target>17</maven.compiler.target>
>
>
> <pulsar.broker.compiler.release>${maven.compiler.target}</pulsar.broker.compiler.release>
> <pulsar.client.compiler.release>8</pulsar.client.compiler.release>
>
> I'm unsure what JavaVersion required and recommend for Pulsar broker.
>
> IIUC https://github.com/apache/pulsar/pull/15264 and PIP-156 (
> https://github.com/apache/pulsar/issues/15207) are ongoing efforts to run
> pulsar broker with JDK17.
>
> Best,
> tison.
>
> [1] https://github.com/apache/pulsar/pull/16163
>