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.

Sign up for GitHub

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

Jump to bottom

[Camel 3.12] Subclasses of ServiceCallConfiguration should not need to be registered for reflection #2955

Closed
ppalaga opened this issue Jul 28, 2021 · 2 comments
Closed

[Camel 3.12] Subclasses of ServiceCallConfiguration should not need to be registered for reflection #2955

ppalaga opened this issue Jul 28, 2021 · 2 comments
Assignees
@jamesnetherton
Labels
housekeeping
Milestone
2.4.0

Comments

@ppalaga
Copy link
Contributor

ppalaga commented Jul 28, 2021

We should investigate whether the subclasses of org.apache.camel.model.cloud.ServiceCallConfiguration can be introspected without using Java reflection in Camel.

Once this is solved in Camel, org.apache.camel.quarkus.core.cloud.deployment.CloudProcessor.reflectiveClasses(BuildProducer<ReflectiveClassBuildItem>, CombinedIndexBuildItem) can be removed.
@ppalaga
Copy link
Contributor Author

ppalaga commented Jul 28, 2021

Blocked by https://issues.apache.org/jira/browse/CAMEL-16828

@ppalaga ppalaga changed the title Subclasses of ServiceCallConfiguration should not need to be registered for reflection [Camel 3.12] Subclasses of ServiceCallConfiguration should not need to be registered for reflection Jul 29, 2021
@ppalaga
Copy link
Contributor Author

ppalaga commented Jul 29, 2021

The blocking issue was resolved. Thanks @davsclaus!

@jamesnetherton jamesnetherton self-assigned this Oct 4, 2021
@jamesnetherton jamesnetherton added this to the 2.4.0 milestone Oct 4, 2021
jamesnetherton added a commit to jamesnetherton/camel-quarkus that referenced this issue Oct 4, 2021
@jamesnetherton
Remove reflective class registration for subclasses of ServiceCallCon…
18eedef 
…figuration

Fixes apache#2955
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jamesnetherton jamesnetherton

Labels
housekeeping
Projects
None yet
Milestone
2.4.0
Development

No branches or pull requests
2 participants
@ppalaga @jamesnetherton