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: New fetch job function. #1241

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

Conversation

TheNeedForSleep
Copy link

@TheNeedForSleep TheNeedForSleep commented Nov 20, 2024

fixes #1242

New fetch job function:

  • handles on update conflict
  • works with same lock different priority task order
  • inner select uses index properly

Does not support the aborting status!

Successful PR Checklist:

  • Tests
    • (not applicable?)
  • Documentation
    • (not applicable?)

PR label(s):

New fetch job function:
- handles on update conflict
- works with same lock different priority task order
- inner select uses index properly
@TheNeedForSleep TheNeedForSleep requested a review from a team as a code owner November 20, 2024 14:20
@github-actions github-actions bot added the PR type: bugfix 🕵️ Contains bug fix label Nov 20, 2024
Fetch job with retry after max retries get any doable job instead
@TheNeedForSleep
Copy link
Author

The code is not ready to be merged!

@TkTech
Copy link

TkTech commented Dec 3, 2024

The code is not ready to be merged!

You can mark the PR as a draft to make sure it doesn't get accidentally merged :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR type: bugfix 🕵️ Contains bug fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

procrastinate_fetch_job can end up being slow
3 participants