Hi Mark,
On 16/12/2019 16:10, Mark Thomas wrote:
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() ?
ModuleReader::list is intended to list all resources in a module, this
includes all class files and non-class files.
So not much else to say on this because enumerating the contents of a
module is completely independent on
whether preview features are enabled or not.
I hope that helps ?
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.
Thanks for the list of open issues , we are monitoring.
Rgds,Rory
Mark
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
--
Rgds, Rory O'Donnell
Quality Engineering Manager
Oracle EMEA, Dublin, Ireland
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
