Thank you!

I'll test the JDK 13 compiled server with JDK 8 both on branch-3.6 and
branch-3.5 tomorrow.
If I succeed, then I will resolve
https://issues.apache.org/jira/browse/ZOOKEEPER-3215 by explaining the
fixes and set the fixedVersions to 3.5.8, 3.6.2 and 3.7.0.

Mate

On Wed, Apr 29, 2020 at 7:36 PM Andor Molnar <an...@apache.org> wrote:

> Required 2 simple patches from David Mollitor:
>
>
> https://github.com/apache/zookeeper/commit/1c5d135d74f16275876c024401dc2de92909b20a
>
> https://github.com/apache/zookeeper/commit/591e1fdbb98a151e003066305e1cc1fd27dcfb66
>
> …in order to get Jute’s pom.xml in sync with master.
> I manually cherry-picked and pushed the commits into 3.6 and 3.5 branches
> instead of creating PRs. I’m also updating Jiras.
>
> Please let me know if you have any concerns.
>
> Andor
>
>
>
>
> > On 2020. Apr 29., at 19:18, Andor Molnar <an...@apache.org> wrote:
> >
> > Got it.
> >
> > zookeeper-jute uses and old version of Maven compiler plugin which
> doesn’t support the jdk-release-flag profile.
> > I’ll create a new patch soon.
> >
> > Andor
> >
> >
> >
> >> On 2020. Apr 29., at 11:48, Andor Molnar <an...@apache.org> wrote:
> >>
> >> I keep looking today.
> >> Chris, if you have some cycles, we could use your brain here.
> >>
> >> There were a good deal of Maven improvements around recently. Why
> haven’t we backported them to 3.5?
> >> It’s an active release, no code changes, build improvements are always
> good, …
> >>
> >> Andor
> >>
> >>
> >>
> >>> On 2020. Apr 29., at 11:29, Andor Molnar <an...@apache.org> wrote:
> >>>
> >>> You’re right. The patch doesn’t work on 3.5 for some reason.
> >>> There might be some other Maven patch missing.
> >>>
> >>> Andor
> >>>
> >>>
> >>>
> >>>> On 2020. Apr 29., at 10:18, Andor Molnar <an...@apache.org> wrote:
> >>>>
> >>>> Checking.
> >>>>
> >>>>
> >>>>> On 2020. Apr 29., at 9:41, Szalay-Bekő Máté <
> szalay.beko.m...@gmail.com> wrote:
> >>>>>
> >>>>> I merged the change and tried to test this locally on branch-3.5 but
> >>>>> haven't succeed so far.
> >>>>>
> >>>>> Steps:
> >>>>> - checkout branch-3.5 (I already merged the PR)
> >>>>> - build ZK from source with openjdk 13.0.2 (I used maven 3.6.3 and
> "git
> >>>>> clean -xdf && mvn clean install -DskipTests")
> >>>>> - run server with JDK 8 (I don't have Oracle JDK, but tried
> "8.0.252-zulu",
> >>>>> "8.0.242-zulu" and "8.0.212-zulu" with sdkman.io, these should be
> OpenJDK
> >>>>> builds)
> >>>>>
> >>>>> I got "java.lang.NoSuchMethodError:
> >>>>> java.nio.ByteBuffer.clear()Ljava/nio/ByteBuffer;" exception in the
> server
> >>>>> logs right when I started the server.
> >>>>>
> >>>>> Could someone verify this behaviour on branch-3.5?
> >>>>>
> >>>>> There might be some other differences between branch-3.5 and master.
> Or
> >>>>> maybe OpenJDK 8 behaves differently than the Oracle JDK used by
> Andor for
> >>>>> testing?
> >>>>> I'll try to dig deeper later.
> >>>>>
> >>>>> On Tue, Apr 28, 2020 at 3:26 PM Andor Molnar <an...@apache.org>
> wrote:
> >>>>>
> >>>>>> https://github.com/apache/zookeeper/pull/1340
> >>>>>>
> >>>>>>
> >>>>>>
> >>>>>>> On 2020. Apr 28., at 15:04, Andor Molnar <an...@apache.org> wrote:
> >>>>>>>
> >>>>>>> I verified the patch, it works perfectly.
> >>>>>>>
> >>>>>>> Steps:
> >>>>>>> - build ZK from source with openjdk 13.0.2
> >>>>>>> - run server with Oracle 1.8.0_201
> >>>>>>> - connect zkcli with both java versions
> >>>>>>> - run some smoke tests
> >>>>>>>
> >>>>>>> It worked perfectly with master and failed with 3.5.7 with the
> >>>>>> “NoSuchMethodError” as mentioned in the ticket.
> >>>>>>>
> >>>>>>> I’ll backport the patch to 3.6 and 3.5 and mark the ticket
> resolved.
> >>>>>>>
> >>>>>>> Thanks again.
> >>>>>>>
> >>>>>>> Andor
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>
> >>>>>>>> On 2020. Apr 28., at 13:27, Andor Molnar <an...@apache.org>
> wrote:
> >>>>>>>>
> >>>>>>>> Great. I’ll take care of that.
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>
> >>>>>>>>> On 2020. Apr 28., at 13:08, Enrico Olivelli <eolive...@gmail.com
> >
> >>>>>> wrote:
> >>>>>>>>>
> >>>>>>>>> Il Mar 28 Apr 2020, 10:48 Andor Molnar <an...@apache.org> ha
> scritto:
> >>>>>>>>>
> >>>>>>>>>> Really?
> >>>>>>>>>> That patch is a general solution and actually looks really
> promising
> >>>>>> to
> >>>>>>>>>> me. I’ll check.
> >>>>>>>>>>
> >>>>>>>>>
> >>>>>>>>> I forgot that in that patch option 2 is already implemented
> >>>>>>>>> https://github.com/apache/zookeeper/pull/1269
> >>>>>>>>> We could port it to 3.6 branch and to branch 3.5
> >>>>>>>>>
> >>>>>>>>> Enrico
> >>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>> Thanks Chris.
> >>>>>>>>>>
> >>>>>>>>>> Andor
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>> On 2020. Apr 28., at 9:34, Enrico Olivelli <
> eolive...@gmail.com>
> >>>>>> wrote:
> >>>>>>>>>>>
> >>>>>>>>>>> Il Mar 28 Apr 2020, 02:47 Christopher <ctubb...@apache.org> ha
> >>>>>> scritto:
> >>>>>>>>>>>
> >>>>>>>>>>>> Option 2 was implemented by me already as part of my pull
> request
> >>>>>> for
> >>>>>>>>>>>> https://issues.apache.org/jira/browse/ZOOKEEPER-3739
> >>>>>>>>>>>> This change was applied to 3.6.1 and 3.7.0.
> >>>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>> This is not the problem but the problem with ByteBuffer.
> >>>>>>>>>>>
> >>>>>>>>>>> Enrico
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>>> If this didn't fix it, I'd have to look into it further. But,
> the
> >>>>>>>>>>>> option 1 doesn't look too bad to me.
> >>>>>>>>>>>>
> >>>>>>>>>>>> On Mon, Apr 27, 2020 at 1:16 PM Enrico Olivelli <
> >>>>>> eolive...@gmail.com>
> >>>>>>>>>>>> wrote:
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> Optional 2 is the best.
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> I have fallen into this problem while preparing 3.6.1 rc. I
> was
> >>>>>> using
> >>>>>>>>>>>> jdk14
> >>>>>>>>>>>>> and my binaries wouldn't run in jdk8
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> Enrico
> >>>>>>>>>>>>>
> >>>>>>>>>>>>> Il Lun 27 Apr 2020, 18:58 Andor Molnar <an...@apache.org> ha
> >>>>>> scritto:
> >>>>>>>>>>>>>
> >>>>>>>>>>>>>> Hi folks,
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> https://issues.apache.org/jira/browse/ZOOKEEPER-3215
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> has come up recently again and I’m thinking of the right
> way to
> >>>>>>>>>> resolve
> >>>>>>>>>>>>>> this. Given that we have some Maven experts in the house my
> first
> >>>>>>>>>>>> question
> >>>>>>>>>>>>>> would be: can we just resolve this with some Maven magic?
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Option #1
> >>>>>>>>>>>>>> Apply https://github.com/apache/zookeeper/pull/760 on
> master
> >>>>>> branch
> >>>>>>>>>> to
> >>>>>>>>>>>>>> make explicit casts which will hint the compiler to use the
> right
> >>>>>>>>>>>> method.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Option #2
> >>>>>>>>>>>>>> Detect the Java compiler with Maven and add “—release 8”
> switch
> >>>>>> to all
> >>>>>>>>>>>>>> Java compilers with version 9+.
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> I really want to resolve this with some automated solution,
> >>>>>> because
> >>>>>>>>>>>>>> changing the source code seems to me ugly and fragile. Do
> we have
> >>>>>> any
> >>>>>>>>>>>> other
> >>>>>>>>>>>>>> option?
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>> Regards,
> >>>>>>>>>>>>>> Andor
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>>>
> >>>>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>>>
> >>>>>>>>
> >>>>>>>
> >>>>>>
> >>>>>>
> >>>>
> >>>
> >>
> >
>
>

Reply via email to