Skip to content

Make contextual RAG worker count configurable - #14431

Open
Brainsucker92 wants to merge 2 commits into
onyx-dot-app:mainfrom
Brainsucker92:fix/contextual-rag-worker-concurrency-main
Open

Make contextual RAG worker count configurable#14431
Brainsucker92 wants to merge 2 commits into
onyx-dot-app:mainfrom
Brainsucker92:fix/contextual-rag-worker-concurrency-main

Conversation

@Brainsucker92

@Brainsucker92 Brainsucker92 commented Sep 2, 2026

Copy link
Copy Markdown

Makes MAX_CONTEXTUAL_RAG_WORKERS configurable via the environment while preserving 128 as the default. This allows Onyx deployments using single-concurrency Ollama runners to reduce contextual-RAG request concurrency.


Summary by cubic

Makes contextual RAG and image worker counts configurable via MAX_CONTEXTUAL_RAG_WORKERS and MAX_IMAGE_WORKERS, keeping 128 and 16 as defaults. Previously both were hardcoded, which forced single-concurrency Ollama runners to handle too many concurrent requests; deployments can now lower these values to reduce concurrency. Values below 1 are clamped to 1.

Written for commit 0c96bce. Summary will update on new commits.

Review in cubic

@Brainsucker92
Brainsucker92 requested a review from a team as a code owner September 2, 2026 14:42
@greptile-apps

greptile-apps Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

PR author is not in the allowed authors list.

@Brainsucker92
Brainsucker92 force-pushed the fix/contextual-rag-worker-concurrency-main branch from cd74dfb to 14ab178 Compare September 2, 2026 14:44

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 1 file (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread backend/onyx/indexing/indexing_pipeline.py Outdated
@Brainsucker92
Brainsucker92 force-pushed the fix/contextual-rag-worker-concurrency-main branch from 14ab178 to 7f0361d Compare September 2, 2026 16:35

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 1 file (changes from recent commits).

Tip: Review your code locally with the cubic CLI to iterate faster.
Tip: cubic used a learning from your PR history. Let your coding agent read cubic learnings directly with the cubic MCP.

Re-trigger cubic

Comment thread backend/onyx/indexing/indexing_pipeline.py Outdated
Brainsucker92 and others added 2 commits September 4, 2026 22:41
Co-authored-by: cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
@Brainsucker92
Brainsucker92 force-pushed the fix/contextual-rag-worker-concurrency-main branch from 24c7cf4 to 0c96bce Compare September 4, 2026 20:41
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.

1 participant