Boy - it seems it's a mistake for the JDK to have Record in the java.lang package. Putting it in a different package would fix this. I wonder if we should file a bug or bring it up on the Amber list?
-Jordan > On Mar 27, 2020, at 11:26 AM, Enrico Olivelli <[email protected]> wrote: > > Let me file and INFRA issue and a patch for jute > > Stay tuned > > Enrico > > Il Ven 27 Mar 2020, 17:03 Patrick Hunt <[email protected]> ha scritto: > >> Confirmed locally with oracle jdk 14 and zk trunk. >> >> Patrick >> >> On Fri, Mar 27, 2020 at 6:19 AM Enrico Olivelli <[email protected]> >> wrote: >> >>> Il giorno gio 26 mar 2020 alle ore 23:45 Patrick Hunt >>> <[email protected]> ha scritto: >>>> >>>> Seems the new JEP 359 record feature is added to jdk14 as a preview and >>>> it's introduced a regression wrt our "Record" >>>> https://openjdk.java.net/jeps/359 >>>> >>>> So two things then - we should disambiguate our Record and see why the >>>> jenkins job is not seeing this... odd. >>> >>> Maybe Jenkins has an early version of JDK14 without records support >>> >>> having a mvn -v on jenkins will help >>> >>> Can anyone try locally ? >>> You can download the jdk and use it just by unpacking the tar.gz file, >>> no need to "install" it >>> >>> Enrico >>> >>>> >>>> Patrick >>>> >>>> >>>> On Thu, Mar 26, 2020 at 3:26 PM Enrico Olivelli <[email protected]> >>> wrote: >>>> >>>>> Patrick >>>>> you are right >>>>> it looks like it is using "/home/jenkins/tools/java/latest14" >>>>> >>>>> this is my maven version info: >>>>> >>>>> [eolivelli@localhost target]$ mvn -v >>>>> Apache Maven 3.6.3 (cecedd343002696d0abb50b32b541b8a6ba2883f) >>>>> Maven home: /home/eolivelli/dev/maven >>>>> Java version: 14, vendor: AdoptOpenJDK, runtime: >>>>> /home/eolivelli/dev/jdk-14+36 >>>>> Default locale: en_US, platform encoding: UTF-8 >>>>> OS name: "linux", version: "5.5.10-200.fc31.x86_64", arch: "amd64", >>>>> family: "unix" >>>>> >>>>> we should add some "mvn -v" to be executed as a pre build step >>>>> >>>>> Enrico >>>>> >>>>> Il giorno gio 26 mar 2020 alle ore 23:22 Patrick Hunt >>>>> <[email protected]> ha scritto: >>>>>> >>>>>> The jenkins job for jdk14 is passing - any ideas why you are seeing >>>>>> different? Is the jenkins job setup incorrectly? >>>>>> >>>>>> >>>>> >>> >> https://builds.apache.org/view/Z/view/ZooKeeper/job/zookeeper-master-maven-jdk14/ >>>>>> >>>>>> Patrick >>>>>> >>>>>> >>>>>> On Thu, Mar 26, 2020 at 3:13 PM Enrico Olivelli < >> [email protected] >>>> >>>>> wrote: >>>>>> >>>>>>> Hi, >>>>>>> it looks like ZK cannot be build on JDK14 due to a small source >>>>>>> compatibility issue. >>>>>>> The error is below. >>>>>>> >>>>>>> The fix is trivial, we just only have to explicitly import the >> full >>>>>>> classname of "Record" >>>>>>> >>>>>>> Enrico >>>>>>> >>>>>>> both interface org.apache.jute.Record in org.apache.jute and >> class >>>>>>> java.lang.Record in java.lang match >>>>>>> [ERROR] >>>>>>> >>>>> >>> >> /home/eolivelli/dev/zookeeper/zookeeper-jute/target/generated-sources/java/org/apache/zookeeper/proto/GetMaxChildrenResponse.java:25: >>>>>>> error: reference to Record is ambiguous >>>>>>> [ERROR] public class GetMaxChildrenResponse implements Record { >>>>>>> >>>>> >>> >>
