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

Allow serializing complex objects in JobDataMaps when using Quartz extension #33534

Conversation

machi1990
Copy link
Member

@machi1990 machi1990 commented May 22, 2023

Fixes #30834

@machi1990 machi1990 changed the title Allow serializing complex objects in JobDataMaps Allow serializing complex objects in JobDataMaps when using Quartz extension May 22, 2023
@machi1990 machi1990 requested a review from mkouba May 22, 2023 20:52
@machi1990 machi1990 linked an issue May 22, 2023 that may be closed by this pull request
@quarkus-bot

This comment has been minimized.

@machi1990 machi1990 force-pushed the quartz/allow-serializing-complex-object-in-job-data branch from fbbdac5 to 36218cf Compare May 23, 2023 09:58
@machi1990 machi1990 requested a review from mkouba May 23, 2023 09:59
Copy link
Contributor

@mkouba mkouba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@mkouba mkouba added the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 23, 2023
@machi1990 machi1990 force-pushed the quartz/allow-serializing-complex-object-in-job-data branch from 36218cf to 6107453 Compare May 23, 2023 10:28
@machi1990
Copy link
Member Author

Thanks for the review @mkouba
I've added the StoreType#RAM enum type in the latest push.

@quarkus-bot
Copy link

quarkus-bot bot commented May 23, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@machi1990 machi1990 merged commit 6e3a7f1 into quarkusio:main May 23, 2023
@quarkus-bot quarkus-bot bot added kind/enhancement New feature or request and removed triage/waiting-for-ci Ready to merge when CI successfully finishes labels May 23, 2023
@quarkus-bot quarkus-bot bot added this to the 3.2 - main milestone May 23, 2023
@machi1990 machi1990 deleted the quartz/allow-serializing-complex-object-in-job-data branch May 23, 2023 12:23
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 this pull request may close these issues.

Quartz configuration useProperties in application.properties file
2 participants