You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@camel.apache.org by "Łukasz Dywicki (Created JIRA)" <ji...@apache.org> on 2012/04/06 16:59:24 UTC
[jira] [Created] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Support cdi annotations in beans / camel-cdi refactor.
------------------------------------------------------
Key: CAMEL-5145
URL: https://issues.apache.org/jira/browse/CAMEL-5145
Project: Camel
Issue Type: New Feature
Reporter: Łukasz Dywicki
Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
3. Implement CdiInjector.
There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (Updated JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: CAMEL-5145-2.diff
New path uses method introduced in latest deltaspike commits.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (Updated JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: CAMEL-5145.diff
Patch for CDI injections and refactor camel-cdi.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (Commented JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13250736#comment-13250736 ]
Łukasz Dywicki commented on CAMEL-5145:
---------------------------------------
Hey,
Deltaspike is planed to release 0.2 version soon, probably this or next week. Once issue I reported them will be closed I'll attach path to fix snapshot dependency.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13258863#comment-13258863 ]
Claus Ibsen commented on CAMEL-5145:
------------------------------------
Lukasz you must mark [x] in grant license when you attach files. Otherwise we cannot accept your patches.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: CAMEL-5145-4.patch
Fix java.util.logging statements printed to console.
Fix checkstyle warnings.
Fix duplicated slf4j binding dependencies.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145-4.patch, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (Updated JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: CAMEL-5145-3.diff
Change version of deltaspike to released (currently during vote).
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (Updated) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Claus Ibsen updated CAMEL-5145:
-------------------------------
Fix Version/s: 2.10.0
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13255404#comment-13255404 ]
Claus Ibsen commented on CAMEL-5145:
------------------------------------
Thanks Lukasz. Let us know when the release is in maven central, then we can apply the patch.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13251319#comment-13251319 ]
Claus Ibsen commented on CAMEL-5145:
------------------------------------
Sounds good. Your patch looks great, so looking forward to the released deltaspike.
Do you mind help with CDI documentation?
Maybe a new page from the user guide about CDI? Like we have for blueprint etc.
Using CDI with Camel
http://camel.apache.org/user-guide.html
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13255404#comment-13255404 ]
Claus Ibsen commented on CAMEL-5145:
------------------------------------
Thanks Lukasz. Let us know when the release is in maven central, then we can apply the patch.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (Commented) (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13249215#comment-13249215 ]
Claus Ibsen commented on CAMEL-5145:
------------------------------------
Thanks Lukasz. Is there an ETA on a new release of deltaspike, so we can avoid SNAPSHOT dependencies.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13259662#comment-13259662 ]
Łukasz Dywicki commented on CAMEL-5145:
---------------------------------------
Claus,
All patches can be included in ASF works. At least the management page for attachements shows green "YES" for all of them. Btw. Do not include CAMEL-5145.diff it's outdated. Use 2nd and 3rd patch only.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Resolved] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Claus Ibsen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Claus Ibsen resolved CAMEL-5145.
--------------------------------
Resolution: Fixed
Assignee: Łukasz Dywicki
Create new tickets for other cdi improvements
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Assignee: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145-5.patch, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Hadrian Zbarcea (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13261013#comment-13261013 ]
Hadrian Zbarcea commented on CAMEL-5145:
----------------------------------------
Lukasz, patches applied with thanks. If the docs are up to date please feel free to mark this issue as resolved.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145-5.patch, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: (was: CAMEL-5145-4.patch)
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145-5.patch, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Hadrian Zbarcea (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13260157#comment-13260157 ]
Hadrian Zbarcea commented on CAMEL-5145:
----------------------------------------
Lukasz, thanks for the patch(es). One thing you could try before we close this is to change the logging for org.apache.webbeans to file while running the tests, so that successful tests are silent.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Commented] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (Commented JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13251424#comment-13251424 ]
Łukasz Dywicki commented on CAMEL-5145:
---------------------------------------
I'll take care about CDI docs too. There is bunch of features which are present for now in camel-spring but still missing in camel-cdi. I wish to implement at least part of them.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
[jira] [Updated] (CAMEL-5145) Support cdi annotations in beans /
camel-cdi refactor.
Posted by "Łukasz Dywicki (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/CAMEL-5145?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Łukasz Dywicki updated CAMEL-5145:
----------------------------------
Attachment: CAMEL-5145-5.patch
Merge previous patch with latest changes on trunk. Fixes logging issue with open web beans. Fixes duplicated slf4j binding for weld profile.
> Support cdi annotations in beans / camel-cdi refactor.
> ------------------------------------------------------
>
> Key: CAMEL-5145
> URL: https://issues.apache.org/jira/browse/CAMEL-5145
> Project: Camel
> Issue Type: New Feature
> Reporter: Łukasz Dywicki
> Labels: cdi
> Fix For: 2.10.0
>
> Attachments: CAMEL-5145-2.diff, CAMEL-5145-3.diff, CAMEL-5145-4.patch, CAMEL-5145-5.patch, CAMEL-5145.diff
>
>
> Currently camel-cdi supports lookups by name. If user would like to use CDI-annotated class then he will not get injections done. During implementation of this feature I've done some more changes:
> 1. Refactor test classes to rely on deltaspike container API. It allows to run tests with Open WebBeans (by default) or JBoss Weld (optional). There is no direct dependency to any implementation of CDI. Only public interfaces. Also I've removed long assert from test case and put equal check to test POJO.
> 2. Remove BeanProvider and BeanManagerProvider, as they are already implemented in deltaspike-core (which is independent from CDI implementation)
> 3. Implement CdiInjector.
> There is a issue DELTASPIKE-151 created by me today to support also Map as result of lookup which needs to be applied. If it will be not included then I'll attach another patch which will implement necessary code in camel-cdi.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira