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

Make timeout in @TransactionConfiguration configurable #15752

Closed
marcelhanser opened this issue Mar 16, 2021 · 1 comment · Fixed by marcelhanser/quarkus#1
Closed

Make timeout in @TransactionConfiguration configurable #15752

marcelhanser opened this issue Mar 16, 2021 · 1 comment · Fixed by marcelhanser/quarkus#1
Labels
kind/enhancement New feature or request

Comments

@marcelhanser
Copy link

Description

With the @TransactionConfiguration annotation you can set a specific timeout for a transaction. Thats quit helpful for some situations, Unforntunately the timeout given in seconds cannot be externalized as property. So the only way to change the specific timeout is to build/deploy a new version. I would like to configure the timeout without a new build.

One solution would be to do it similar to the @scheduled annotation. Something like:
@TransactionConfiguration(timeoutFromConfigProperty="{....}")

@marcelhanser marcelhanser added the kind/enhancement New feature or request label Mar 16, 2021
geoand added a commit to geoand/quarkus that referenced this issue Mar 18, 2021
geoand added a commit to geoand/quarkus that referenced this issue Mar 18, 2021
marcelhanser added a commit to marcelhanser/quarkus that referenced this issue May 19, 2021
Allow setting transaction timeout from a config property
marcelhanser added a commit to marcelhanser/quarkus that referenced this issue May 19, 2021
Allow setting transaction timeout from a config property
marcelhanser referenced this issue in marcelhanser/quarkus May 19, 2021
@marcelhanser
Copy link
Author

Hmm, seems like i did something wrong. Actually i just wanted to implement the caching remark

@marcelhanser marcelhanser reopened this May 19, 2021
marcelhanser pushed a commit to marcelhanser/quarkus that referenced this issue May 19, 2021
marcelhanser referenced this issue in marcelhanser/quarkus May 19, 2021
marcelhanser referenced this issue in marcelhanser/quarkus May 19, 2021
marcelhanser referenced this issue in marcelhanser/quarkus May 20, 2021
marcelhanser referenced this issue in marcelhanser/quarkus May 20, 2021
Resolves: quarkusio#15752
gsmet referenced this issue in marcelhanser/quarkus Jun 11, 2021
gsmet referenced this issue in marcelhanser/quarkus Jun 11, 2021
Resolves: quarkusio#15752
@gsmet gsmet closed this as completed in 33690e0 Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment