do we really need corba anymore? Am Mo., 20. Sept. 2021 um 18:53 Uhr schrieb Zowalla, Richard < [email protected]>:
> I think that there are two different ideas / thoughts in here: > > (a) JDK / JRE 17 Runtime: Be able to deploy and operate a TomEE with a > Java 17 environment. This might be solvable by adding JVM options (as > mentioned by Thiago and Vicente) or doing some rework of the proxy > factory (looks like a solution without the JVM flags as mentioned by > Jon) > > (b) _Compiling_ TomEE with JDK 11+ > > For (b) and Corba: > > I had worked on it a in 2020 and had two drafts. In May, we had some > issues with a force push to master, which closed / killed all open PRs, > which render the diff nearly useless. The drafts: > > - https://github.com/apache/tomee/pull/713 (use byte code generation at > build time -> drawbacks as mentioned by Romain) > > - https://github.com/apache/tomee/pull/792 (using SPI to lookup a corba > integration module, which can be compiled with Java 8 and is not > included in the main tomee project. I guess it would require some > cleanup / update and the open tasks adressed. > > > PR 721 had some open questions, which I couldn't resolve at the time it > was created. It might be worth to pick up the work on a J11+ build. The > mail discussion is located here [1]. > > Gruss > Richard > > > > [1] https://www.mail-archive.com/[email protected]/msg14097.html > > Am Montag, dem 20.09.2021 um 10:28 -0600 schrieb Memo Díaz Solis: > > I just tested by myself (regarding the CORBA stuff) and of course > > got the > > issue you mentioned. Is there any alternative besides the glassfish > > jar you > > guys might know? or do we want to remove CORBA at all? > > <dependency> > > <groupId>org.glassfish.packager</groupId> > > <artifactId>glassfish-corba-omgapi</artifactId> > > <version></version> > > </dependency> > > > > El lun, 20 sept 2021 a las 9:42, Vicente Rossello (< > > [email protected]>) > > escribió: > > > > > Hi, > > > > > > My two cents... > > > > > > I've tried the examples, don't know how to fix it permanently, but > > > I've > > > noticed two things: > > > * Lombok should be upgraded to 1.18.20 to be able to compile with > > > JDK16+ ( > > > https://projectlombok.org/changelog ) > > > * OrbFactory does not compile in JDK11+ as well. I don't really > > > know if > > > CORBA is still used by anyone > > > > > > I've been running tomee in JDK17 for some time now, everything > > > works fine > > > (there are 4 add-opens in catalina.bat / catalina.sh). > > > > > > On Mon, Sep 20, 2021 at 1:21 PM Zowalla, Richard < > > > [email protected]> wrote: > > > > > > > Thanks for the additional hint. It fixes the test, but I still > > > > think, > > > > that we should make a permanent (code-based) fix ;) > > > > > > > > I have created a related JIRA: > > > > > > > > - https://issues.apache.org/jira/browse/TOMEE-3795 > > > > > > > > In addition, it looks like > > > > > > > > - https://issues.apache.org/jira/browse/TOMEE-2744 > > > > > > > > is another thingy for J16 / J17. > > > > > > > > Perhaps we should create a tracking issue (JDK17 compatibility) > > > > to > > > > aggregate the different issues to tackle. > > > > > > > > Am Montag, dem 20.09.2021 um 07:45 -0300 schrieb Thiago Henrique > > > > Hupner: > > > > > The easy fix is to replace the illegal access to - - add-opens > > > > > and - > > > > > - > > > > > add-exports. > > > > > > > > > > The exception message includes which one is required. > > > > > > > > > > Em seg., 20 de set. de 2021 07:34, Zowalla, Richard < > > > > > [email protected]> escreveu: > > > > > > > > > > > I think that's the approach taken by the guys at deltaspike > > > > > > in > > > > > > Thomas' > > > > > > link [1] (and the related PR in [2]). > > > > > > > > > > > > [1] https://issues.apache.org/jira/browse/DELTASPIKE-1432 > > > > > > [2] https://github.com/apache/deltaspike/pull/119 > > > > > > > > > > > > Am Montag, dem 20.09.2021 um 11:09 +0100 schrieb Jonathan > > > > > > Gallimore: > > > > > > > I had a very quick look - I wonder if we simply need to > > > > > > > move to > > > > > > > MethodHandles.Lookup.defineClass(): > > > > > > > > > > > https://docs.oracle.com/javase/9/docs/api/java/lang/invoke/MethodHandles.Lookup.html#defineClass-byte:A- > > > > > > > in the LocalBeanProxyFactory. There's likely a couple of > > > > > > > other > > > > > > > libraries > > > > > > > that TomEE uses that do similar things, such as OWB and > > > > > > > OpenJPA, > > > > > > > so > > > > > > > we'll > > > > > > > need to check that they do the right thing in Java 17 as > > > > > > > well. > > > > > > > > > > > > > > Jon > > > > > > > > > > > > > > On Mon, Sep 20, 2021 at 9:08 AM Zowalla, Richard < > > > > > > > [email protected]> wrote: > > > > > > > > > > > > > > > Hi ladies, > > > > > > > > > > > > > > > > I wanted to start a discussion about TomEE and its Java > > > > > > > > 17 > > > > > > > > compatibility. > > > > > > > > > > > > > > > > I did some tests in one of our demo projects It looks > > > > > > > > like that > > > > > > > > " > > > > > > > > --illegal-access=permit" was removed in Java 17, which > > > > > > > > leads to > > > > > > > > some InaccessibleObjectException [1]. > > > > > > > > > > > > > > > > Does anybody know, if there are solutions / workarounds > > > > > > > > for it > > > > > > > > in > > > > > > > > Java > > > > > > > > 17 or do we need to fix it? If so: how? ;) > > > > > > > > > > > > > > > > I bet, that many other frameworks / containers will run > > > > > > > > in this > > > > > > > > issue, > > > > > > > > so maybe someone already thought about a possible > > > > > > > > solution? > > > > > > > > Discussion > > > > > > > > seems to be "hot" [2]. > > > > > > > > > > > > > > > > Gruss > > > > > > > > Richard > > > > > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > https://gist.github.com/rzo1/5c1d0724a700e262a2e2824ded9f32ab > > > > > > > > [2] > > > > > > > > https://twitter.com/delabassee/status/1438446396865224705 > > > > > > > > > > > > > > > > > > > > > > > > > -- > Richard Zowalla, M.Sc. > Research Associate, PhD Student | Medical Informatics > > Hochschule Heilbronn – University of Applied Sciences > Max-Planck-Str. 39 > D-74081 Heilbronn > phone: +49 7131 504 6791 (zur Zeit nicht via Telefon erreichbar) > mail: [email protected] > web: https://www.mi.hs-heilbronn.de/ >
