[ 
https://issues.apache.org/jira/browse/PIVOT-885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13577488#comment-13577488
 ] 

Sandro Martini commented on PIVOT-885:
--------------------------------------

Karel, thank you very much for this contribution ... if there will not be 
objections from other Pivot PMC Members, we could start to look at it for Pivot 
2.1 .
The problem is that it contains a lot of classes, interfaces, etc, so really we 
need some time to better look at it and start playing with it, aligning with 
the rest of Pivot code base. I think this would be a great addition, but we 
must handle it carefully.

My primary concern here is:
- if there is something that we can do (maybe even now, in the 2.0.x 
maintenance branch) to be able to use this as is before being integrated in 
Pivot ... if you have some suggestion, please tell us. For sure I already have 
some idea even to improve some existing Pivot classes.
- of course we have to look even on how to integrate with other (de facto 
standards) for Dependency Injection (Spring, Java EE 6 DI, etc), to abstract 
only one time things in Pivot, and then maybe integrate/provide only yours in 
Pivot ... 
For example we could even think on providing its implementation under 
apache-extras and keep in Pivot only a minimal (and generic) version of it, 
because maybe some people doesn't need/want these features integrated if not 
using them (for example direct RMI calls are a not-so-common usage now, but I 
see value in this, so please don't get me wrong).

Anyway, depending on what the community and PMC say we need to make changes to 
the initial version (in attach here), but for sure we'll need some hint from 
you.

I'll give you some initial feedback soon, I already started to look at it (but 
roughly).
Thanks again for now.

                
> Add support for dependecy injection and RMI
> -------------------------------------------
>
>                 Key: PIVOT-885
>                 URL: https://issues.apache.org/jira/browse/PIVOT-885
>             Project: Pivot
>          Issue Type: New Feature
>            Reporter: Karel Hübl
>            Assignee: Sandro Martini
>             Fix For: 2.1
>
>         Attachments: extensions-sample.zip, pivot-extensions.zip
>
>
> As pivot application devoloper, I would like to have support for dependency 
> injection in Pivot. My requirements are:
> - support for field injection (at least)
> - support for @PostConstruct methods
> - support for @RuntimeScoped classes - only one instance may be constructed 
> during Runtime. Usefull when one needs share injected instances.
> - in relevant cases, inject dependencies also into namespace.
> I implemented prototype of this functionality, source codes including unit 
> tests and sample application are included. The dependency injection is 
> implemented in proposed org.apache.pivot.beans.injection package.
> Please consider adding this functionality in future Pivot release.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to