-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[persistence.jpa] Upgrade openjpa and derby #16591
Conversation
18e6771
to
324b3a4
Compare
@holgerfriedrich maybe you can build a KAR and post it in the community forums, so someone can test it? I don't know anyone who is actively using it. I also couldn't find anyone that raised an issue lately. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, let's wait for some test results or merge it and get some live results as i don't expect anyone available to test it prior to a (milestone) release.
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]>
As i don't expect tests from here. Let's see what some milage on snapshot / next milestone brings before next stable. It would be fairly easy to revert if an unexpected issue shows up. |
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]> Signed-off-by: Paul Smedley <[email protected]>
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]> Signed-off-by: Patrik Gfeller <[email protected]>
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]>
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]>
Support Java 21 - Upgrade openjpa from 3.2.2 to 4.0.0 - Upgrade derby from 10.14.2.0 to 10.16.1.1 (last Java 17 release) - Migrate javax.el to jakarta.el-api 6.0.0-RC1 - Revert workaround for Java 21, openhab#16612 Signed-off-by: Holger Friedrich <[email protected]> Signed-off-by: Ciprian Pascu <[email protected]>
This updates all base libraries of persistence.java. Goal is to support compiling and running with Java 21 (currently we compile this directory with target release Java 17).
Note that Derby 10.16.x.x is the last release built for Java 17. It is still affected by CVE-2022-46337. Using version 10.17.x.x which contains a fix is not yet possible until we drop Java 17 support. Derby project did not release fixed version compiled for Java 17.
As I do not use this binding, I need help testing this version. Unfortunately, the binding does not contain test cases.
Refs: openhab/openhab-distro#1590