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

Jakarta - EE 9 - Native - Quartz and JTA #27065

Closed
gsmet opened this issue Aug 1, 2022 · 3 comments · Fixed by #28261
Closed

Jakarta - EE 9 - Native - Quartz and JTA #27065

gsmet opened this issue Aug 1, 2022 · 3 comments · Fixed by #28261
Assignees
@gsmet
Labels
area/jakarta area/scheduler
Milestone
2.13.4.Final

Comments

@gsmet
Copy link
Member

gsmet commented Aug 1, 2022

Quartz has a JTA layer and it relies on the javax classes.

Unfortunately the latest Quartz version dates from October 2019 so I wonder if it's still maintained.
There are a few commits that are more recent in the repository though.

I don't know if we are supporting this JTA setup in Quarkus.
@quarkus-bot
Copy link

quarkus-bot bot commented Aug 1, 2022

/cc @machi1990, @mkouba

@mkouba
Copy link
Contributor

mkouba commented Aug 1, 2022

I found this comment: "2.4.0 should be released in the near future.". However, the 2.4.0 milestone does not seem to contain anything Jakarta-related.

I don't know if we are supporting this JTA setup in Quarkus.

I think that quarkus.quartz.store-type=JDBC_TX does rely on JTA...

@gsmet
Copy link
Member Author

gsmet commented Sep 28, 2022

I will reuse the same technique @Sanne used for the Db2 JDBC so we can have this fixed while we wait for a Quartz release.

gsmet added a commit that referenced this issue Sep 28, 2022
@gsmet
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
3a4af32 
Fixes #27065
@gsmet gsmet self-assigned this Sep 28, 2022
@gsmet gsmet mentioned this issue Sep 28, 2022
Merged
@quarkus-bot quarkus-bot bot added this to the 2.14 - main milestone Sep 28, 2022
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@gsmet @igorregis
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
7c60d2f 
Fixes quarkusio#27065
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@gsmet @igorregis
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
fbfbefc 
Fixes quarkusio#27065
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 16, 2022
@gsmet @igorregis
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
fe9f997 
Fixes quarkusio#27065
igorregis pushed a commit to igorregis/quarkus that referenced this issue Oct 17, 2022
@gsmet @igorregis
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
78b5a7a 
Fixes quarkusio#27065
@gsmet gsmet modified the milestones: 2.14 - main, 2.13.4.Final Oct 24, 2022
gsmet added a commit that referenced this issue Oct 25, 2022
@gsmet
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
74cfc80 
Fixes #27065

(cherry picked from commit 3a4af32)
zakkak pushed a commit to zakkak/quarkus that referenced this issue Nov 15, 2022
@gsmet @zakkak
Jakarta - Rewrite Quartz bytecode to jakarta.* packages
75918ce 
Fixes quarkusio#27065

(cherry picked from commit 3a4af32)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gsmet gsmet

Labels
area/jakarta area/scheduler
Projects
Quarkus Roadmap/Planning
Status: Done
Milestone
2.13.4.Final
Development

Successfully merging a pull request may close this issue.

More Jakarta progress - Quartz, RESTEasy 6.2, SmallRye Reactive Messaging upgrade... gsmet/quarkus
2 participants
@mkouba @gsmet