On 14/12/2019 16:48, Rory O'Donnell wrote: > > Hi Mark, > > *Per the JDK 14 schedule , we are now in Rampdown Phase One* > > *Please advise if you have found any issues while testing the latest > Early Access build.
Rory, The addition of java.lang.Record triggered as unit test failure (as it is meant to) and we have updated the EL optimisation code that depends on knowing the names of all the classes in the java.lang package to take account of this new class. The tests are passing again. However, the tests were run without enabling preview features. With that in mind should the Record class have been returned when enumerating the classes via ModuleReader.list() ? The relevant source code is: https://github.com/apache/tomcat/blob/master/test/javax/el/TestImportHandlerStandardPackages.java#L146 No other test failures were observed when building an running the full test suite with EA27. Passes a basic smoke test (WebSocket snake game over TLS). We do have a couple of unresolved bugs open at the moment: - https://bugs.openjdk.java.net/browse/JDK-8234808 - https://bugs.openjdk.java.net/browse/JDK-8233619 I'm currently working with Xuelei Fan on JDK-8233619. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org
