the concrete interceptor-strategies (like TransactionStrategy) are part of our spi. your suggestion would mean that we would need to move them as well (= remove them from the spi). def. -1 for that because i know several users who are using them. i really don't get the issue you have with a simple marker interface (after we have it for 7 years - including codi).
btw. there are users out there who re-use InterceptorStrategy for their internal interceptor-strategies (of their own libs). regards, gerhard 2018-04-26 6:41 GMT+02:00 Romain Manni-Bucau <[email protected]>: > Still means it doesnt have to be in the API right? > > Le 26 avr. 2018 00:44, "Gerhard Petracek" <[email protected]> a écrit : > > > #1 with cdi 1.0 (or to be more concrete: owb for cdi 1.0) you can't get > rid > > of pre-configured interceptors (that's why we introduced the > > interceptor-strategy concept initially). > > #2 e.g. TransactionStrategy has benefits beyond that (a public example is > > the usage in the ds-data-module) > > > > regards, > > gerhard > > > > > > > > 2018-04-25 6:58 GMT+02:00 Romain Manni-Bucau <[email protected]>: > > > > > I get it but it means we add a layer on top of interceptor for > > > pluggability. This is actually built in in CDI so not really needed. > > > > > > Also the hierarchy point is fine but should be per type of strategy and > > > therefore we dont need a generic one in the api. > > > > > > As a user if i use DS and an interceptor, do i need to impl this public > > > api? Never normally so this sounds more misleading or reinventing the > > wheel > > > than anything else for me. > > > > > > That said we can move it in our impl modules to keep the feature but > > still > > > a clean api. > > > > > > Le 24 avr. 2018 23:21, "Gerhard Petracek" <[email protected]> a > > écrit : > > > > > > > a concrete example: > > > > @Transactional > > > > > > > > -> > > > > @Interceptor is on TransactionalInterceptor whereas > InterceptorStrategy > > > is > > > > the marker interface for the strategies (and not the interceptor) - > in > > > this > > > > case TransactionStrategy. > > > > > > > > (to quickly get an overview of all interceptor-strategies you just > need > > > to > > > > open the hierarchy-view for InterceptorStrategy and you have > everything > > > you > > > > need with one step...) > > > > > > > > regards, > > > > gerhard > > > > > > > > > > > > > > > > 2018-04-24 22:35 GMT+02:00 Romain Manni-Bucau <[email protected] > >: > > > > > > > > > Hmm not sure i get it, annotations are hard to browse in IDE? Is it > > > what > > > > it > > > > > addresses? > > > > > > > > > > Le 24 avr. 2018 21:10, "Gerhard Petracek" <[email protected]> a > > > > écrit : > > > > > > > > > > > hi romain, > > > > > > > > > > > > not really. 1 interceptor could have n strategies as candidates > > (e.g. > > > > see > > > > > > TransactionStrategy for which we provide multiple implementations > > > > > > out-of-the-box). > > > > > > that's the whole concept. the marker interfaces is just to find > all > > > > > > strategies in a project easily. > > > > > > we have it since 02/2011 (back then it was codi) and a lot of > > users > > > > are > > > > > > using it (during the dev. process) and i haven't heard about any > > > > concern > > > > > > (from users). > > > > > > > > > > > > regards, > > > > > > gerhard > > > > > > > > > > > > > > > > > > > > > > > > 2018-04-24 19:31 GMT+02:00 Romain Manni-Bucau < > > [email protected] > > > >: > > > > > > > > > > > > > Le 24 avr. 2018 19:18, "Gerhard Petracek" < > [email protected]> > > a > > > > > > écrit : > > > > > > > > > > > > > > it was always just a marker-interface to list all > > > > > interceptor-strategies > > > > > > > easily. > > > > > > > > > > > > > > > > > > > > > But if it is just interceptors, doesnt @Interceptor fulfills > that > > > > > > already? > > > > > > > > > > > > > > My only concern is exposing it in api to user where it is > > actually > > > a > > > > > dead > > > > > > > interface. > > > > > > > > > > > > > > > > > > > > > regards, > > > > > > > gerhard > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2018-04-24 13:47 GMT+02:00 Thomas Andraschko < > > > > > > [email protected] > > > > > > > >: > > > > > > > > > > > > > > > basically +1 > > > > > > > > but its still used currently > > > > > > > > > > > > > > > > > > > > > > > > 2018-04-23 11:46 GMT+02:00 Romain Manni-Bucau < > > > > [email protected] > > > > > >: > > > > > > > > > > > > > > > > > Hi guys, > > > > > > > > > > > > > > > > > > Do we still need InterceptorStrategy? > > > > > > > > > > > > > > > > > > If not, can we deprecate it and remove it from our built-in > > > > > > > interceptors? > > > > > > > > > > > > > > > > > > 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> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
