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

Add job priority support and prioritize user initiated indexing requests #2051

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

habdelra
Copy link
Contributor

@habdelra habdelra commented Jan 15, 2025

This PR add job priority support by allowing you to configure workers that will only process jobs greater than or equal to a particular priority. This allows us to designate workers that will only process user initiated indexing requests (as well as workers that will process any jobs system or user initiated).

This PR uses this new mechanism to set user initiated indexing jobs (incremental indexing and from-scratch indexing related to realm creation) to use the higher priority workers. This will ensure that workers are available to process user initiated index mutations while the realm server spins up and performs background indexing on all the realms on the server.

Paired with PR https://github.com/cardstack/infra/pull/571

Copy link

Host Test Results

    1 files  ±0      1 suites  ±0   21m 25s ⏱️ -14s
729 tests ±0  727 ✔️ ±0  2 💤 ±0  0 ±0 
734 runs  ±0  732 ✔️ ±0  2 💤 ±0  0 ±0 

Results for commit c82b5e4. ± Comparison against base commit d6d07da.

@habdelra habdelra requested a review from a team January 15, 2025 20:03
@lukemelia
Copy link
Contributor

For the future, this would have been easier to review if the private -> # changes were in their own commit.

@habdelra habdelra merged commit 9ab9f58 into main Jan 16, 2025
61 checks passed
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