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

Class loader issues in AvroRpcTest with Quarkus 2.0.0.Alpha3 #2651

Closed
ppalaga opened this issue May 24, 2021 · 4 comments · Fixed by #2859
Closed

Class loader issues in AvroRpcTest with Quarkus 2.0.0.Alpha3 #2651

ppalaga opened this issue May 24, 2021 · 4 comments · Fixed by #2859
Assignees
Labels
Milestone

Comments

@ppalaga
Copy link
Contributor

ppalaga commented May 24, 2021

A variant of #2514

quarkus.test.flat-class-path=true is a workaround

@ppalaga ppalaga added this to the 2.0.0 milestone May 24, 2021
@ppalaga ppalaga added the test label May 24, 2021
@ppalaga ppalaga changed the title Class loader issues in AvroRpcTest with Quarkus 2.0.0.Alpha1 Class loader issues in AvroRpcTest with Quarkus 2.0.0.Alpha3 May 24, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 24, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 24, 2021
ppalaga added a commit to ppalaga/camel-quarkus that referenced this issue May 26, 2021
…h Quarkus 2.0.0.Alpha3"

This reverts commit 406058c.
@ppalaga ppalaga modified the milestones: 2.0.0, 2.0.0-M2 Jun 4, 2021
@JiriOndrusek JiriOndrusek self-assigned this Jun 10, 2021
@ppalaga
Copy link
Contributor Author

ppalaga commented Jun 10, 2021

Related: quarkusio/quarkus#17463

@JiriOndrusek
Copy link
Contributor

Problem is caused by issue in jetty. Wrong classloader is set in jetty. Issue was reported as jetty/jetty.project#6399

@ppalaga
Copy link
Contributor Author

ppalaga commented Jun 18, 2021

I wonder what we can do for 2.0.0 that will happen before Camel 3.11? Could you @JiriOndrusek perhaps check the feasibility of having the Undertow based org.apache.avro.ipc.Server impl. here in Camel Quarkus for now? We could do it properly once we have the necessary changes in Camel.

@JiriOndrusek
Copy link
Contributor

Change in Camel is proposed as https://issues.apache.org/jira/browse/CAMEL-16740 (PR is created - apache/camel#5714)

@ppalaga I'll try to find a way of fixing it in meantime in camel-quarkus only, as you suggested.

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

Successfully merging a pull request may close this issue.

2 participants