Hi Romain,

Well... that change make me possible to continue create my singleton
startup like it:

import javax.annotation.PostConstruct;
import javax.annotation.PreDestroy;
import javax.ejb.Singleton;
import javax.ejb.Startup;

@Singleton
@Startup
public class KernelShutdownListener {

    @PostConstruct
    private void startup() {
        System.out.println("ALOHA STARTUP");
    }

    @PreDestroy
    private void shutdown() {
        System.out.println("ALOHA SHUTDOWN");
    }
}

It was not possible without that change, as you can see here:
Caused by: java.lang.IllegalStateException: Cannot get Unsafe.defineClass
    at
org.apache.webbeans.proxy.AbstractProxyFactory$4.run(AbstractProxyFactory.java:812)
    at
org.apache.webbeans.proxy.AbstractProxyFactory$4.run(AbstractProxyFactory.java:801)
    at java.base/java.security.AccessController.doPrivileged(Native Method)
    at
org.apache.webbeans.proxy.AbstractProxyFactory.initializeUnsafe(AbstractProxyFactory.java:800)
    at
org.apache.webbeans.proxy.AbstractProxyFactory.<init>(AbstractProxyFactory.java:91)
    at
org.apache.webbeans.proxy.InterceptorDecoratorProxyFactory.<init>(InterceptorDecoratorProxyFactory.java:77)
    at
org.apache.webbeans.config.WebBeansContext.<init>(WebBeansContext.java:153)
    at
org.apache.webbeans.config.WebBeansContext.<init>(WebBeansContext.java:116)
    at
org.apache.openejb.cdi.ThreadSingletonServiceImpl.initialize(ThreadSingletonServiceImpl.java:167)
    at org.apache.openejb.cdi.CdiBuilder.build(CdiBuilder.java:41)
    at
org.apache.openejb.assembler.classic.Assembler.createApplication(Assembler.java:963)
    ... 17 more
Caused by: java.lang.NoSuchMethodException:
sun.misc.Unsafe.defineClass(java.lang.String, [B, int, int,
java.lang.ClassLoader, java.security.ProtectionDomain)
    at java.base/java.lang.Class.getDeclaredMethod(Class.java:2476)
    at
org.apache.webbeans.proxy.AbstractProxyFactory$4.run(AbstractProxyFactory.java:807)
    ... 27 more

05-Nov-2018 15:15:33.110 SEVERE [localhost-startStop-2]
jdk.internal.reflect.NativeMethodAccessorImpl.invoke Error deploying web
application archive [/tmp/apache-tomee-plume-7.0.6-TT.2/webapps/ROOT.war]
 java.lang.IllegalStateException: ContainerBase.addChild: start:
org.apache.catalina.LifecycleException: Failed to start component
[StandardEngine[Catalina].StandardHost[localhost].StandardContext[]]
    at
org.apache.catalina.core.ContainerBase.addChildInternal(ContainerBase.java:758)
    at
org.apache.catalina.core.ContainerBase.addChild(ContainerBase.java:730)
    at org.apache.catalina.core.StandardHost.addChild(StandardHost.java:734)
    at org.apache.catalina.startup.HostConfig.deployWAR(HostConfig.java:985)
    at
org.apache.catalina.startup.HostConfig$DeployWar.run(HostConfig.java:1857)
    at
java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:515)
    at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264)
    at
java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
    at
java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
    at java.base/java.lang.Thread.run(Thread.java:834)


Em seg, 5 de nov de 2018 às 15:24, Romain Manni-Bucau <rmannibu...@gmail.com>
escreveu:

> Hi Daniel,
>
> Do you know which part implies the "need", this commit was not suppose to
> change anything since the code was already supporting java 11, it was just
> intended to prepare potential change later on.
> Did a small change in the behavior was introduced - if so it was not
> intended from my part?
>
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> |  Blog
> <https://rmannibucau.metawerx.net/> | Old Blog
> <http://rmannibucau.wordpress.com> | Github <
> https://github.com/rmannibucau> |
> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
> <
> https://www.packtpub.com/application-development/java-ee-8-high-performance
> >
>
>
> Le lun. 5 nov. 2018 à 19:20, Daniel Cunha <daniels...@apache.org> a écrit
> :
>
> > Hi Folks,
> >
> > we got an issue with Java 11 and TomEE 7.0.x and we need the backport for
> > it:
> > https://github.com/apache/openwebbeans/pull/13
> >
> > Please take a look at those changes and let me know what do you think.
> >
> > Thank you.
> > --
> > Daniel "soro" Cunha
> > https://twitter.com/dvlc_
> >
>


-- 
Daniel "soro" Cunha
https://twitter.com/dvlc_

Reply via email to