Skip to content

docs(eval): enrichment concurrency is near-linear (7.8× at concurrency 8) with a parallel backend - #118

Merged
njfio merged 1 commit into
mainfrom
docs/enrichment-concurrency-proof
Jul 23, 2026
Merged

njfio merged 1 commit into
mainfrom
docs/enrichment-concurrency-proof

Conversation

@njfio

@njfio njfio commented Jul 23, 2026

Copy link
Copy Markdown
Owner

Proves the async write-path enrichment concurrency (merged in #117) actually delivers — the earlier real-7B run showed only 1.1× because ollama serialized the requests, not because of the code.

Re-ran the same --deferred-enrichment path (same conv-0, same 8-way enrich_pending) against a mock OpenAI-compatible endpoint that serves requests concurrently (fixed 1s/call) — a genuinely parallel-capable backend:

path time
synchronous (Inline) ingest 844.7 s
deferred: raw write 2.3 s
deferred: enrich (concurrency 8) 106.3 s
total speedup 7.8×

7.8× at concurrency 8 (~97.5% efficiency). The two measurements bracket the truth:

  • real 7B ollama (default serving): 1.1× — backend-serialized.
  • parallel backend: 7.8× — near-linear.

So the async code's concurrency is real and near-linear; the realized wall-clock speedup equals the LLM backend's effective serving parallelism. Docs-only; the real-LLM number remains the honest end-to-end figure for a single local model.

🤖 Generated with Claude Code

…y 8) with a parallel backend

Isolates harness concurrency from the LLM backend: re-ran the deferred-enrichment
path against a concurrent mock endpoint (fixed 1s/call). Synchronous 844.7s vs
deferred 108.6s = 7.8x (~97.5% efficiency at concurrency 8). Brackets the earlier
real-7B result (1.1x, ollama-serialized): the async concurrency is real and
near-linear; realized speedup = the backend's effective serving parallelism.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@njfio
njfio merged commit 5d321af into main Jul 23, 2026
17 of 22 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.

1 participant