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 >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>>> >>>>>> >>>> >>>> >> >>
