i understand, no worries ;)
*Romain Manni-Bucau* *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* *Blog: **http://rmannibucau.wordpress.com/*<http://rmannibucau.wordpress.com/> *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* *Github: https://github.com/rmannibucau* 2013/3/4 Thomas Hug <[email protected]> > Good point, thnx - working with the injection points might do the trick. > RE demo, not sure how far I'll get the next few days (new little family > member who's currently the boss of my schedule ;) but will try to spend > some time on it. > > On Mon, Mar 4, 2013 at 10:59 AM, Romain Manni-Bucau > <[email protected]>wrote: > > > since you already have an extension you could "spy" delegates at startup > > too no? > > > > PS: with Jean-Louis Monteiro we'll do a JUG this month on CDI/DS/TomEE > (not > > in detail but just an overview). This kind of extension would be great to > > show > > > > *Romain Manni-Bucau* > > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > > *Blog: **http://rmannibucau.wordpress.com/*< > > http://rmannibucau.wordpress.com/> > > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > > *Github: https://github.com/rmannibucau* > > > > > > > > 2013/3/4 Thomas Hug <[email protected]> > > > > > Agree, having this working in SE envs would be nice. > > > > > > This is the data container which is related to the currently executed > > query > > > [1]. The main reason for making this request scope injectable was the > > usage > > > for base DAOs or if a client wants to add its own DAO code - here [2] > the > > > call gets delegated to a concrete implementation. > > > > > > Passing the query context data into CDI Query controlled code is not > > really > > > a problem, for client code having something injectable seemed to me the > > > most elegant solution. But there are certainly other variants in order > to > > > get rid of the request scope (e.g. having a contract on the > > > DelegateQueryHandler interface - open for other suggestions). > > > > > > [1] > > > > > > > > > https://github.com/ctpconsulting/query/blob/deltaspike/impl/src/main/java/com/ctp/cdi/query/handler/CdiQueryInvocationContext.java > > > [2] > > > > > > > > > https://github.com/ctpconsulting/query/blob/deltaspike/impl/src/main/java/com/ctp/cdi/query/builder/DelegateQueryBuilder.java > > > > > > On Sun, Mar 3, 2013 at 1:41 PM, Romain Manni-Bucau < > > [email protected] > > > >wrote: > > > > > > > Basically if the query logic could be scope agnostic it would be > > perfect > > > > for me. Parameters were fine and matched the coomon need. I m not > sure > > i > > > > get what is blocking. Can you detail it a bit please? > > > > Le 3 mars 2013 13:09, "Thomas Hug" <[email protected]> a > > > > écrit : > > > > > > > > > Aha got you. Yes QueryHandler can be moved to application scope. > > > > > > > > > > The other request scoped component ist the > CdiQueryInvocationContext. > > > > Here > > > > > we store a bunch of data which is related to the current query > > > > (parameters, > > > > > query string etc). Originally this was passed as parameter, but got > > > > > injectable for the extension mechanism [1]. Currently on holidays > > > (sorry > > > > > the response delays) so not much time to look into it but happy to > > take > > > > > suggestions :) > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > http://ctpconsulting.github.com/query/1.0.0.Alpha5/extensions.html#extension-delegates > > > > > > > > > > On Fri, Mar 1, 2013 at 8:18 PM, Romain Manni-Bucau < > > > > [email protected] > > > > > >wrote: > > > > > > > > > > > Hmm, > > > > > > > > > > > > Wonder if the scope couldnt be application one. Request sounds > too > > > much > > > > > web > > > > > > to be usable/generic > > > > > > Le 1 mars 2013 19:02, "Thomas Hug" < > [email protected]> > > a > > > > > > écrit : > > > > > > > > > > > > > Fine for me - planned to create a branch with all the > > experimental > > > > APIs > > > > > > > cleared out as an import proposal. > > > > > > > > > > > > > > RE scope, the InvocationHandlers are validated to be normal > > scoped > > > > [1], > > > > > > so > > > > > > > that seemed to me the most appropriate choice - any other > > > > suggestions? > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/DeltaSpike/Mirror/blob/master/deltaspike/core/impl/src/main/java/org/apache/deltaspike/core/impl/invocationhandler/InvocationHandlerBindingExtension.java#L198 > > > > > > > > > > > > > > > > > > > > > On Wed, Feb 27, 2013 at 1:15 PM, Romain Manni-Bucau > > > > > > > <[email protected]>wrote: > > > > > > > > > > > > > > > Hi, > > > > > > > > > > > > > > > > some news of it? > > > > > > > > > > > > > > > > i think it is time to import at least a first version, isn't > > it? > > > > > > > > > > > > > > > > Just a note on current code, i saw QueryHandler (and some > other > > > > > beans) > > > > > > > has > > > > > > > > a request scope. Wonder if such a scope means something for a > > > > generic > > > > > > > (not > > > > > > > > web) lib. > > > > > > > > > > > > > > > > wdyt? > > > > > > > > > > > > > > > > *Romain Manni-Bucau* > > > > > > > > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > > > > > > > > *Blog: **http://rmannibucau.wordpress.com/*< > > > > > > > > http://rmannibucau.wordpress.com/> > > > > > > > > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > > > > > > > > *Github: https://github.com/rmannibucau* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2013/2/20 Romain Manni-Bucau <[email protected]> > > > > > > > > > > > > > > > > > guy you rocks! > > > > > > > > > > > > > > > > > > effectively i missed it when i looked. > > > > > > > > > > > > > > > > > > *Romain Manni-Bucau* > > > > > > > > > *Twitter: @rmannibucau <https://twitter.com/rmannibucau>* > > > > > > > > > *Blog: **http://rmannibucau.wordpress.com/*< > > > > > > > > http://rmannibucau.wordpress.com/> > > > > > > > > > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > > > > > > > > > *Github: https://github.com/rmannibucau* > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > 2013/2/20 Thomas Hug <[email protected]> > > > > > > > > > > > > > > > > > >> Thnx Romain. If with proxy stuff you refer to the DS > > > > > > > > >> InvocationHandlerBinding, that's already integrated :) > > > > > > > > >> > > > > > > > > >> RE pagination, it's maybe not that obvious, but there's > more > > > > > > advanced > > > > > > > > >> support in QueryResult [1][2], which is more of a fluent > API > > > > > > approach, > > > > > > > > but > > > > > > > > >> should behave similar to a PageRequest (the query > execution > > is > > > > > > delayed > > > > > > > > >> until the getResult() methods on QueryResult is called). > > Hope > > > > that > > > > > > > fits > > > > > > > > to > > > > > > > > >> what you had in mind. > > > > > > > > >> > > > > > > > > >> [1] > > > > > > > > >> > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/ctpconsulting/query/blob/deltaspike/api/src/main/java/com/ctp/cdi/query/QueryResult.java > > > > > > > > >> [2] > > > > > > > > >> > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://github.com/ctpconsulting/query/blob/deltaspike/impl/src/test/java/com/ctp/cdi/query/QueryResultTest.java > > > > > > > > >> > > > > > > > > >> On Wed, Feb 20, 2013 at 10:04 PM, Romain Manni-Bucau > > > > > > > > >> <[email protected]>wrote: > > > > > > > > >> > > > > > > > > >> > Hi Thomas, > > > > > > > > >> > > > > > > > > > >> > importing proxy stuff would be the first great stuff. > > > > > > > > >> > > > > > > > > > >> > i'd like to see some pagination API (like PageRequest > etc > > in > > > > > > spring > > > > > > > > >> world) > > > > > > > > >> > which are more user friendly that @FirstResult and > > > > @MaxResult). > > > > > > > > >> > > > > > > > > > >> > wdyt? > > > > > > > > >> > > > > > > > > > >> > *Romain Manni-Bucau* > > > > > > > > >> > *Twitter: @rmannibucau <https://twitter.com/rmannibucau > >* > > > > > > > > >> > *Blog: **http://rmannibucau.wordpress.com/*< > > > > > > > > >> > http://rmannibucau.wordpress.com/> > > > > > > > > >> > *LinkedIn: **http://fr.linkedin.com/in/rmannibucau* > > > > > > > > >> > *Github: https://github.com/rmannibucau* > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > 2013/2/20 Thomas Hug <[email protected]> > > > > > > > > >> > > > > > > > > > >> > > CDI Query has now a completely Solder-free POM. Feel > > free > > > to > > > > > > drop > > > > > > > > >> > comments > > > > > > > > >> > > on the API - some parts might actually be fully > excluded > > > > (e.g. > > > > > > > home > > > > > > > > >> and > > > > > > > > >> > > critera APIs). > > > > > > > > >> > > > > > > > > > > >> > > On Tue, Feb 12, 2013 at 10:00 PM, Mark Struberg < > > > > > > > [email protected]> > > > > > > > > >> > wrote: > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > Thanks Thomas, great news! > > > > > > > > >> > > > > > > > > > > > >> > > > LieGrue, > > > > > > > > >> > > > strub > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > > >> > > > >________________________________ > > > > > > > > >> > > > > From: Thomas Hug <[email protected]> > > > > > > > > >> > > > >To: "[email protected]" < > > > > > > > > >> > > > [email protected]> > > > > > > > > >> > > > >Sent: Tuesday, February 12, 2013 4:10 PM > > > > > > > > >> > > > >Subject: Re: cdi-query, no news? > > > > > > > > >> > > > > > > > > > > > > >> > > > >FYI, I've started to "de-solderize" CDI Query and > > move > > > > > things > > > > > > > to > > > > > > > > >> > depend > > > > > > > > >> > > on > > > > > > > > >> > > > >DS Core [1]: > > > > > > > > >> > > > > > > https://github.com/ctpconsulting/query/tree/deltaspike > > > > > > > > >> > > > > > > > > > > > > >> > > > >Todos: > > > > > > > > >> > > > >[x] Replace ServiceHandler > > > > > > > > >> > > > >[_] Include Property utils > > > > > > > > >> > > > >[_] Replace JBoss Logging > > > > > > > > >> > > > > > > > > > > > > >> > > > >Any feedback welcome. > > > > > > > > >> > > > > > > > > > > > > >> > > > >[1] including this modification > > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/DELTASPIKE-113?focusedCommentId=13576531&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13576531 > > > > > > > > >> > > > > > > > > > > > > >> > > > >On Mon, Feb 11, 2013 at 6:01 PM, Romain Manni-Bucau > > > > > > > > >> > > > ><[email protected]>wrote: > > > > > > > > >> > > > > > > > > > > > > >> > > > >> a review of the API i guess > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> what is missing today is probably the pagination > > > > helpers > > > > > > > > >> > (PageRequest > > > > > > > > >> > > > for > > > > > > > > >> > > > >> instance) > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> but technically all is fine IMO > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> *Romain Manni-Bucau* > > > > > > > > >> > > > >> *Twitter: @rmannibucau < > > > > https://twitter.com/rmannibucau > > > > > >* > > > > > > > > >> > > > >> *Blog: **http://rmannibucau.wordpress.com/*< > > > > > > > > >> > > > >> http://rmannibucau.wordpress.com/> > > > > > > > > >> > > > >> *LinkedIn: ** > > http://fr.linkedin.com/in/rmannibucau* > > > > > > > > >> > > > >> *Github: https://github.com/rmannibucau* > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> 2013/2/11 Jason Porter <[email protected]> > > > > > > > > >> > > > >> > > > > > > > > >> > > > >> > I know I'm playing the necromancer, but please > > > > forgive > > > > > > me. > > > > > > > We > > > > > > > > >> have > > > > > > > > >> > > > >> > InvocationHandler, which looks like it would > work > > > as > > > > a > > > > > > > > >> > > ServiceHandler > > > > > > > > >> > > > >> > substitute. What else is needed to get CDI > Query > > > into > > > > > > > > >> DeltaSpike? > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > On Sun, Nov 18, 2012 at 12:35 PM, Romain > > > Manni-Bucau > > > > > > > > >> > > > >> > <[email protected]>wrote: > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > +1, it is a must have for cdi world > > > > > > > > >> > > > >> > > Le 18 nov. 2012 20:04, "john.d.ament" < > > > > > > > > >> [email protected]> > > > > > > > > >> > a > > > > > > > > >> > > > >> écrit : > > > > > > > > >> > > > >> > > > > > > > > > > >> > > > >> > > > RE ServiceHandler - I was one of those who > > > > > previously > > > > > > > > >> > suggested > > > > > > > > >> > > > >> > bringing > > > > > > > > >> > > > >> > > it > > > > > > > > >> > > > >> > > > over to DeltaSpike, you can see > > DELTASPIKE-113 > > > > and > > > > > > find > > > > > > > > the > > > > > > > > >> > > thread > > > > > > > > >> > > > >> > about > > > > > > > > >> > > > >> > > it > > > > > > > > >> > > > >> > > > from April of this year. At the time, I > was > > > in a > > > > > > > > position > > > > > > > > >> > > where I > > > > > > > > >> > > > >> > > couldn't > > > > > > > > >> > > > >> > > > spend much time on open source > contribution. > > > > I've > > > > > > > > recently > > > > > > > > >> > > > changed > > > > > > > > >> > > > >> > jobs, > > > > > > > > >> > > > >> > > > to > > > > > > > > >> > > > >> > > > something that's going to help me spend > some > > > more > > > > > > time > > > > > > > > with > > > > > > > > >> > the > > > > > > > > >> > > > open > > > > > > > > >> > > > >> > > source > > > > > > > > >> > > > >> > > > community, and believe I can pick it back > up > > if > > > > > we're > > > > > > > > >> ready. > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > If everyone's ok with it I can start a new > > > thread > > > > > for > > > > > > > 113 > > > > > > > > >> or > > > > > > > > >> > > > revive > > > > > > > > >> > > > >> the > > > > > > > > >> > > > >> > > old > > > > > > > > >> > > > >> > > > thread (though it may come through with a > > large > > > > > pile > > > > > > of > > > > > > > > >> dust). > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > Regards, > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > John > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > -- > > > > > > > > >> > > > >> > > > View this message in context: > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > http://apache-deltaspike-incubator-discussions.2316169.n4.nabble.com/cdi-query-no-news-tp4654029p4654036.html > > > > > > > > >> > > > >> > > > Sent from the Apache DeltaSpike Incubator > > > > > Discussions > > > > > > > > >> mailing > > > > > > > > >> > > list > > > > > > > > >> > > > >> > > archive > > > > > > > > >> > > > >> > > > at Nabble.com. > > > > > > > > >> > > > >> > > > > > > > > > > > >> > > > >> > > > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > -- > > > > > > > > >> > > > >> > Jason Porter > > > > > > > > >> > > > >> > http://en.gravatar.com/lightguardjp > > > > > > > > >> > > > >> > > > > > > > > > >> > > > >> > > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > > >> > > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > >
