[jira] [Commented] (PIVOT-885) Add support for dependecy injection and RMI

["Sandro Martini (JIRA)" <ji...@apache.org>]

    [ 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