Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

camel 3.8 - Removed reflection in 9 components #2209

Closed
davsclaus opened this issue Feb 6, 2021 · 3 comments
Closed

camel 3.8 - Removed reflection in 9 components #2209

davsclaus opened this issue Feb 6, 2021 · 3 comments
Assignees
Milestone

Comments

@davsclaus
Copy link
Contributor

In Camel 3.8 we have removed reflection in the producer
https://issues.apache.org/jira/browse/CAMEL-16102

At the following components

camel-atomix
camel-caffeine
camel-consul
camel-corda
camel-ehcache
camel-infinispan
camel-servicenow
camel-spring-redis
camel-web3j

The camel-quarkus extensions of those should be checked if we have any reflection "stuff" we then can remove.

@davsclaus
Copy link
Contributor Author

@jamesnetherton wonder if any of the extensions has some open for reflection which we can remove. We may wait for Lucas PR to merge first to have it work with 3.8 again.

Mind that some of the components above did not use reflection but was slow by doing annotation scanning on methods, which maybe in graal is possible.

@jamesnetherton
Copy link
Contributor

From the list, only caffeine, consul, infinispan & servicenow have native support. None of those extensions have reflective class configuration for any Camel classes, so I think we're in good shape.

I had to add a reflection hack on the camel-master branch for infinispan, but I'll remove before we do the 3.8.0 upgrade since it's no longer needed.

@jamesnetherton jamesnetherton self-assigned this Feb 10, 2021
@jamesnetherton jamesnetherton added this to the 1.7.0 milestone Feb 11, 2021
@jamesnetherton
Copy link
Contributor

I checked and we are reflection free for those extensions. I cleaned up Netty in #2232.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants