Think we didnt test that, thanks for catching that and adding a test Thomas!
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 mar. 16 oct. 2018 à 09:18, Thomas Andraschko <andraschko.tho...@gmail.com> a écrit : > Not sure. I just searched over the tests yesterday and we didn't had tests > for a generic ProcessManagedBean, so i added the new ones. > I will do some more tests. > > Am Di., 16. Okt. 2018 um 08:24 Uhr schrieb Mark Struberg > <strub...@yahoo.de.invalid>: > > > Didn't we have a test for generic types already? > > E.g. ProcessAnnotatedType<Poodle> vs ProcessAnnotatedType<Sheep> > > And one type layer down > > ProcessAnnotatedType<Housingl<Poodle>> vs > > ProcessAnnotatedType<Housing<Sheep>> > > > > If we don't have this kind of test yet then we should add one. > > > > LieGrue, > > strub > > > > > Am 15.10.2018 um 11:23 schrieb Thomas Andraschko < > > andraschko.tho...@gmail.com>: > > > > > > ok... i totally forgot about generic lifecycle events... > > > > > > I will try to just optimize the current NotificationManager, without > > adding > > > a breaking cache :D > > > > > > Am Fr., 12. Okt. 2018 um 16:48 Uhr schrieb Thomas Andraschko < > > > andraschko.tho...@gmail.com>: > > > > > >> ok, will provide a better patch on monday! > > >> > > >> Am Fr., 12. Okt. 2018 um 16:44 Uhr schrieb Romain Manni-Bucau < > > >> rmannibu...@gmail.com>: > > >> > > >>> We can surely clean it up after startup but looks good > > >>> > > >>> Le ven. 12 oct. 2018 17:16, Thomas Andraschko < > > >>> andraschko.tho...@gmail.com> > > >>> a écrit : > > >>> > > >>>> Hi, > > >>>> > > >>>> as you already may know from the TomEE/BVal mailing list, i'm not > > happy > > >>>> with the startup performance since TomEE8. > > >>>> > > >>>> TomEE7 OWB startup ~ 4,5seconds > > >>>> TomEE8 OWB startup ~ 9seconds > > >>>> > > >>>> i finally managed to improve the performance on 2 places. > > >>>> > > >>>> 1) Filter framework classes in BVal on PAT > > >>>> Reduces the startup to ~8seconds > > >>>> > > >>>> 2) Cache obervers for systemEvents in > > >>> NotificationManager#resolveObservers > > >>>> Reduces the starup to ~7 seconds > > >>>> Patch: > > >>>> > https://gist.github.com/tandraschko/de4c72c22e65aaefe041af5a15d110e3 > > >>>> > > >>>> Do you think the patch is ok or could it break something? > > >>>> I would be happy if mark or romain could improve it :D > > >>>> > > >>>> Best regards, > > >>>> Thomas > > >>>> > > >>> > > >> > > > > >