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

feat: expose missedHeartbeatsLimit config through Spring Boot properties (addresses #633) #634

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wallind
Copy link
Contributor

@wallind wallind commented Feb 19, 2025

Brief, plain english overview of your changes here

Exposes the missedHeartbeatsLimit configuration added in #432 through the Spring Boot DbSchedulerProperties.

Fixes

Closes #633.

Reminders

  • Added/ran automated tests
  • Update README and/or examples
  • Ran mvn spotless:apply

cc @kagkarlsson

@wallind wallind changed the title 633: expose missedHeartbeatsLimit config through Spring Boot properties feat:(633) expose missedHeartbeatsLimit config through Spring Boot properties Feb 19, 2025
@wallind wallind changed the title feat:(633) expose missedHeartbeatsLimit config through Spring Boot properties feat: expose missedHeartbeatsLimit config through Spring Boot properties (addresses #633) Feb 19, 2025
@wallind
Copy link
Contributor Author

wallind commented Feb 19, 2025

I made an educated guess how you might like tests updated for this work e7ccf82; if you'd like something different just let me know and I'm happy to adjust.

@wallind wallind marked this pull request as ready for review February 19, 2025 17:44
/**
* How many heart beats can be missed before an execution is considered dead.
*
* <p>Must be greater than 4
Copy link
Owner

Choose a reason for hiding this comment

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

Must be >= 4

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yep yep; addressed in rebased history, see 4c28c7a

Copy link
Owner

@kagkarlsson kagkarlsson left a comment

Choose a reason for hiding this comment

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

Appreciate the contribution! Thanks!

@wallind wallind force-pushed the 633-expose-missedHeartbeatsLimit-config-through-Spring-Boot-properties branch from e7ccf82 to d07ab8b Compare March 8, 2025 17:49
@wallind wallind requested a review from kagkarlsson March 8, 2025 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: Expose missedHeartbeatsLimit through DbSchedulerProperties
2 participants