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

Stability improvements: #30

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

romantitov
Copy link
Contributor

@romantitov romantitov commented Sep 15, 2023

  • Added logic to recover blocked triggers (we had some cases when a job was interrupted but remained blocked until a cluster restart. With the fix such jobs will be recovered automatically)
  • Fixed an issue when a job stopped during execution (this is one of the cases when a job will stay blocked )
  • Implemented IAsyncDisposable instead of IDisposable to improve performance

- Fixed an issue when a job stopped during execution
- Added logic to recover blocked triggers
- Implemented IAsyncDisposable instead of IDisposable to improve performance
@romantitov
Copy link
Contributor Author

Hi @frantisekjandos any plans to complete the PR and update nuget? The improvement is important for us.

@frantisekjandos
Copy link
Collaborator

frantisekjandos commented Sep 22, 2023

Hi @romantitov, please, allow new admin @footcha some time to process your PR. I am no longer involved in development.

@romantitov
Copy link
Contributor Author

Hi @frantisekjandos thanks for the answer. This is a PR to 'Oriflame/cosmosdb-quartznet'. @footcha please let me know what kind of additional permissions I can provide you?

@footcha
Copy link
Member

footcha commented Sep 22, 2023

@romantitov No action is required from your side at the moment. We need to review your PR internally which will require some time in order to allocate people accountable. Thank you for understanding.

@romantitov
Copy link
Contributor Author

Hello @footcha, do you have any updates? Feel free to ask me any questions about the PR.

@kirillburton
Copy link

Woah, that's exactly what we were looking for to deal with our problem too!
Are there any updates here? @footcha

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.

4 participants