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

Reply via email to