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

Quarkus 2.0 #120

Merged
merged 1 commit into from
Jun 4, 2021
Merged

Quarkus 2.0 #120

merged 1 commit into from
Jun 4, 2021

Conversation

loicmathieu
Copy link
Collaborator

No description provided.

@loicmathieu loicmathieu mentioned this pull request Jun 1, 2021
Closed
@loicmathieu loicmathieu marked this pull request as ready for review June 2, 2021 10:29
@loicmathieu loicmathieu force-pushed the feat/quarkus-2 branch 2 times, most recently from e5669b9 to fdf172b Compare June 2, 2021 13:04
@loicmathieu
Copy link
Collaborator Author

This PR is failing due to java.lang.ClassNotFoundException: com.aayushatharva.brotli4j.decoder.DecoderJNI$Wrapper while building the native image.

So it is blocked untill quarkusio/quarkus#17604 is backported to 2.0 and a release is done.

@gsmet
Copy link
Member

gsmet commented Jun 2, 2021

I will release a CR3 tomorrow if all goes well.

@loicmathieu
Copy link
Collaborator Author

Thanks @gsmet with 2.0.0.CR3 all green ;)

@ppalaga you can use the version from main to try with Camel, I didn't publish CR nor snapshot versions yet, maybe this needs to be done someday to easy the integration between us.

@loicmathieu loicmathieu merged commit cdb5ffb into quarkiverse:main Jun 4, 2021
@ppalaga
Copy link
Contributor

ppalaga commented Jun 4, 2021

Yes, testing with s snapshot locally now. A CR release would be handy to be able to test also within Quarkus Platform and within Camel K (depends on Camel Quarkus)

@ppalaga
Copy link
Contributor

ppalaga commented Jun 4, 2021

Our tests passed with your snapshot.

@loicmathieu loicmathieu deleted the feat/quarkus-2 branch June 4, 2021 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@loicmathieu @gsmet @ppalaga