*start any app with the maven embedded plugin Roberto Cortez <radcor...@yahoo.com.invalid> schrieb am Fr., 12. Okt. 2018, 17:21:
> Hi Thomas, > > If you already have an app that you can share with me that reproduces the > issue it would be great. If not I’ll do one here. > > Cheers, > Roberto > > > On 12 Oct 2018, at 15:37, Thomas Andraschko <andraschko.tho...@gmail.com> > wrote: > > > > Yep, it doesn't work there either. > > BVal 1.x doesn't require so much metadata-lookup as 2.x, so therefore the > > overhead is now much bigger in TomEE8.x now > > > > > > > > > > Am Fr., 12. Okt. 2018 um 16:32 Uhr schrieb Roberto Cortez > > <radcor...@yahoo.com.invalid>: > > > >> Did you check if this also happens in TomEE 7.x? > >> > >>> On 12 Oct 2018, at 15:29, Thomas Andraschko < > andraschko.tho...@gmail.com> > >> wrote: > >>> > >>> Yep > >>> 1 min ago i just verified that it only occurs on embedded-maven-plugin > >>> > >>> maybe you have a good idea to fix it > >>> > >>> in meantime i will check if we should enhance the BValCdiFilter. > >>> > >>> Am Fr., 12. Okt. 2018 um 16:25 Uhr schrieb Roberto Cortez > >>> <radcor...@yahoo.com.invalid>: > >>> > >>>> Thomas, you mean that in the same context and execution if you try > >>>> loadClass you get the ClassNotFound but the Class.forName is able to > >> load > >>>> the class? > >>>> > >>>> Cheers, > >>>> Roberto > >>>> > >>>>> On 12 Oct 2018, at 13:44, Romain Manni-Bucau <rmannibu...@gmail.com> > >>>> wrote: > >>>>> > >>>>> ok, means the TCCL can have changed and be wrong here due to a change > >> in > >>>>> the eager boot code which can capture apploader or maven classloader > >>>>> instead of the mojo classloader. > >>>>> > >>>>> 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 ven. 12 oct. 2018 à 14:31, Thomas Andraschko < > >>>> andraschko.tho...@gmail.com> > >>>>> a écrit : > >>>>> > >>>>>> i'm on tomee-embedded-maven-plugin > >>>>>> > >>>>>> Am Fr., 12. Okt. 2018 um 14:09 Uhr schrieb Romain Manni-Bucau < > >>>>>> rmannibu...@gmail.com>: > >>>>>> > >>>>>>> hmm the loader is the parent loader (container one) and it must > work, > >>>> are > >>>>>>> you in a plain tomee or another flavor like tomee-embedded or > other? > >>>>>>> Class.forName uses the caller classloader which should be the same > >> here > >>>>>> in > >>>>>>> the best case > >>>>>>> > >>>>>>> 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 ven. 12 oct. 2018 à 12:38, Thomas Andraschko < > >>>>>>> andraschko.tho...@gmail.com> > >>>>>>> a écrit : > >>>>>>> > >>>>>>>> I dont think thats related to bval, class.forName works fine > >>>>>>>> > >>>>>>>> Romain Manni-Bucau <rmannibu...@gmail.com> schrieb am Fr., 12. > Okt. > >>>>>>> 2018, > >>>>>>>> 12:20: > >>>>>>>> > >>>>>>>>> Le ven. 12 oct. 2018 11:10, Thomas Andraschko < > >>>>>>>> andraschko.tho...@gmail.com > >>>>>>>>>> > >>>>>>>>> a écrit : > >>>>>>>>> > >>>>>>>>>> Hi, > >>>>>>>>>> > >>>>>>>>>> found a bug, which is related to a slow startup as discussed in > >> the > >>>>>>>> other > >>>>>>>>>> thread. > >>>>>>>>>> > >>>>>>>>>> See: https://issues.apache.org/jira/browse/TOMEE-2258 > >>>>>>>>>> the lookup: > >>>>>> loader.loadClass("org.apache.openejb.bval.BValCdiFilter") > >>>>>>>>>> fails with a ClassNotFound > >>>>>>>>>> > >>>>>>>>>> could someone check this? > >>>>>>>>>> We could easily change it to Class.forName - but not sure what > is > >>>>>> the > >>>>>>>>>> correct classloader. > >>>>>>>>>> > >>>>>>>>> > >>>>>>>>> Openejb core one so should be good. I guess it is a transitive > >> issue > >>>>>>> or a > >>>>>>>>> breaking change in bval we can fix > >>>>>>>>> > >>>>>>>>> > >>>>>>>>> > >>>>>>>>>> We should definitily fix such bugs as the startup with TomEE8, > >>>>>>> compared > >>>>>>>>> to > >>>>>>>>>> TomEE7, is double as slow. > >>>>>>>>>> > >>>>>>>>>> Best regards, > >>>>>>>>>> Thomas > >>>>>>>>>> > >>>>>>>>> > >>>>>>>> > >>>>>>> > >>>>>> > >>>> > >>>> > >> > >> > >
