Good news guys :) Issue was actually a side effect of OWB-1241 (so more related to qualifiers than types). In other words: use >= 2.0.5 and you are good to go :) Sorry for the noise
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 dim. 12 août 2018 à 19:01, Romain Manni-Bucau <rmannibu...@gmail.com> a écrit : > Investigated a bit more, we already have this kind of impl > (using > org.apache.webbeans.container.InjectionResolver#findByParameterizedType) > but it doesn't resolve generics. > I'll check if I can enhance current logic. > > 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 dim. 12 août 2018 à 18:33, Romain Manni-Bucau <rmannibu...@gmail.com> > a écrit : > >> Cause you measured it? I have no app where it is useful once the scanning >> configured and it just breaks multiple apps OOTB and goes against CDI do we >> need a way to fix it. Can be a fastMatchUntilAmbiguous or so but we need >> sthg ;) >> >> Le dim. 12 août 2018 17:34, Thomas Andraschko < >> andraschko.tho...@gmail.com> a écrit : >> >>> Then -1 to set it as default :D >>> >>> >>> Romain Manni-Bucau <rmannibu...@gmail.com> schrieb am So., 12. Aug. >>> 2018, >>> 15:36: >>> >>> > Without. Any generic injection is broken in any project and de are not >>> spec >>> > compliant without that flag. >>> > >>> > Not sure it does worth having our default for most apps but i know for >>> very >>> > big ones it saves some boot time. >>> > >>> > Le dim. 12 août 2018 13:13, Mark Struberg <strub...@yahoo.de.invalid> >>> a >>> > écrit : >>> > >>> > > hmm which test is that? >>> > > >>> > > And does deltaspike break with or without the fastMatch flag? >>> > > >>> > > LieGrue, >>> > > strub >>> > > >>> > > > Am 10.08.2018 um 14:23 schrieb Romain Manni-Bucau < >>> > rmannibu...@gmail.com >>> > > >: >>> > > > >>> > > > Up, >>> > > > >>> > > > just realized it affects deltaspike-data which can't inject >>> > > > EntityRepository<Entity, >>> > > > String> cause of that >>> > > > >>> > > > 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 dim. 22 avr. 2018 à 10:14, Romain Manni-Bucau < >>> > rmannibu...@gmail.com> >>> > > a >>> > > > écrit : >>> > > > >>> > > >> Oki, let me know if we can do it and I would be happy to do the >>> > change. >>> > > >> Alternatively - if we can't, maybe we should introduce that flag >>> per >>> > jar >>> > > >> or extension potentially to enable a "mixed" app (to think if >>> previous >>> > > >> option doesn't become possible). >>> > > >> >>> > > >> >>> > > >> 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 >>> > > > >>> > > >> >>> > > >> 2018-04-22 10:09 GMT+02:00 Mark Struberg >>> <strub...@yahoo.de.invalid>: >>> > > >> >>> > > >>> I'd need to do some tests. But think it's fine. >>> > > >>> >>> > > >>> LieGrue, >>> > > >>> strub >>> > > >>> >>> > > >>>> Am 21.04.2018 um 18:48 schrieb Romain Manni-Bucau < >>> > > >>> rmannibu...@gmail.com>: >>> > > >>>> >>> > > >>>> Hi guys, >>> > > >>>> >>> > > >>>> do we want to switch our fastMatching flag on the 2.x? (think we >>> > cant >>> > > on >>> > > >>>> 1.x) >>> > > >>>> >>> > > >>>> Rational is we would be spec compliant. I know it has some issue >>> > with >>> > > >>> the >>> > > >>>> old way to create producers without generics (or more exactly >>> with a >>> > > >>>> wildcard) but it is not spec compliant too. >>> > > >>>> >>> > > >>>> I encounter that issue (of having the wrong default) writing an >>> > > >>> extension >>> > > >>>> where i register the exact types even for templates (Foo<X> >>> would >>> > have >>> > > >>>> registered Foo<A>, Foo<B>, ... as needed) >>> > > >>>> >>> > > >>>> wdyt? >>> > > >>>> >>> > > >>>> 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 >>> > > >>>> >>> > > >>> >>> > > >>> >>> > > >> >>> > > >>> > > >>> > >>> >>
