Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-05-19 Thread PengHui Li
Hi Heesung, Great work! I support the change. Thanks, Penghui On Thu, May 19, 2022 at 7:46 AM Heesung Sohn wrote: > Hi, > > As a follow-up task here, I have a proposal to update pulsar server default > GC configs. I tried to summarize the details in my fork PR: > https://github.com/heesung-sn/

Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-05-18 Thread Heesung Sohn
Hi, As a follow-up task here, I have a proposal to update pulsar server default GC configs. I tried to summarize the details in my fork PR: https://github.com/heesung-sn/pulsar/pull/1, and It would be great if I could get some early feedback from the community. Thank you, Heesung On Thu, Apr 21,

Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-04-21 Thread Jiuming Tao
+1 Thanks, Tao Jiuming > 2022年4月19日 上午2:25,Heesung Sohn 写道: > > Dear Pulsar dev community, > > My name is Heesung Sohn, and I would like to open a discussion here > about PIP-156: > Build and Run Pulsar Server on Java 17 > . > > > @Nicolò Boschi,

Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-04-20 Thread Heesung Sohn
Hi Lari, Thank you for providing the context about the ongoing work. I added my comments on the email thread and the PR

Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-04-20 Thread Lari Hotari
Hi Heesung, I support this proposal, but I would have expected you to first reply on the email thread started by Nicolo in February https://lists.apache.org/thread/c0k8p9vy5wyp9l70mt980gdy10smx6qb . Nicolo Boschi has been working on Java 17 compatibility for months and also brought this up to

Re: PIP-156: Build and Run Pulsar Server on Java 17

2022-04-20 Thread Nicolò Boschi
Heesung, thanks for the heads up. I added a comment in the PIP. I believe that the compile compatibility can be addressed with another pull and it does not overlap mine. Nicolò Boschi Il giorno lun 18 apr 2022 alle ore 20:25 Heesung Sohn < heesung.s...@streamnative.io> ha scritto: > Dear Pulsa