Let me file and INFRA issue and a patch for jute Stay tuned
Enrico Il Ven 27 Mar 2020, 17:03 Patrick Hunt <ph...@apache.org> ha scritto: > Confirmed locally with oracle jdk 14 and zk trunk. > > Patrick > > On Fri, Mar 27, 2020 at 6:19 AM Enrico Olivelli <eolive...@gmail.com> > wrote: > > > Il giorno gio 26 mar 2020 alle ore 23:45 Patrick Hunt > > <ph...@apache.org> 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 <eolive...@gmail.com> > > 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 > > > > <ph...@apache.org> 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 < > eolive...@gmail.com > > > > > > > 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 { > > > > > > > > > > > > >