rmannibucau commented on a change in pull request #4562:
URL: https://github.com/apache/camel/pull/4562#discussion_r520607787



##########
File path: 
components/camel-cdi/src/main/java/org/apache/camel/cdi/AnnotatedDelegate.java
##########
@@ -21,11 +21,13 @@
 import java.util.HashSet;
 import java.util.Set;
 
+import javax.enterprise.inject.Vetoed;

Review comment:
       Indeed it does not but not because of @vetoed, because it was compiled 
with cdi 2.0 API (bean attributes) so binary is not CDI 1.0 compatible. See 
org.apache.camel.cdi.CdiCamelBeanRepository#findByTypeWithName which uses 
javax.enterprise.inject.spi.BeanAttributes#getName for a single example.
   
   side note for that: pom setup breaks multiimpl setup for java > 8 (java 9 
profile forces weld).




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to