sql_materialize holds the per-session scratch lock for the whole remote source read, blocking all other scratch work
Maintainer thường phản hồi trong vòng 1 ngày
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- sql, typescript
- Lĩnh vực
- databases, performance
Hướng nghiên cứu
Start in packages/opencode/src/altimate/tools/sql-materialize.ts around lines 373-380 and 472-480 to see which work sits inside withScratchConnection, then read exclusive() and the per-session queue in packages/opencode/src/altimate/native/connections/scratch.ts. Done means preflight and the source read run outside the lock, only the write phase takes it, and a SELECT 1 on scratch returns in under a second during a slow materialize. The fix changes concurrency semantics, so confirm the design with maintainers first.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
While one sql_materialize reads from a slow remote source, every other scratch operation in the session (sql_execute on scratch, sql_check, schema_inspect, another sql_materialize from a different source) queues behind it, including calls from sub-agents.
Mechanism
sql_materializerunspreflight()andexecuteSource()against the remote source insidewithScratchConnection(...)(packages/opencode/src/altimate/tools/sql-materialize.ts:373-380).- That callback is
exclusive()(packages/opencode/src/altimate/native/connections/scratch.ts:107), a single promise chain per root session shared by all sub-agents (:51-61,:170). - Only the write phase (
BEGIN/CREATE/INSERT/COMMIT,sql-materialize.ts:472-480) needs the lock. - Reported tool
durationMsincludes queue time, which also misleads timing analysis.
Evidence
Recorded sessions:
- One session was blocked for 1,464 s: three SQLite reads that take 0.6 s alone, issued from 4 parallel sub-agents, all finished together with a multi-minute remote read.
- Other blocks of 706, 404, 376, 129 and 43 s; 10,753 call-seconds of waiting across 7 sessions, none of which produced a correct answer.
- Up to 96 scratch calls in flight at once, completing in bursts; one-row
COUNT(*)queries waited 140-280 s. - In another set of sessions 824 s of queueing across 59 sessions; a 4-row lookup waited 115 s behind an unrelated materialization; a 6-row aggregate took 48.7 s because it queued behind a 48.3 s materialization; an estimated 370 s of wall time lost in 89 generations.
Impact
Time (and, under a time budget, answers): it prevents fan-out from running in parallel and adds directly to budget pressure. Observed with both Claude Opus and gpt-5.6-sol sessions, worst under sub-agent fan-out.
Proposed fix
- Run preflight and the source read outside the lock (stream to a staging file or buffer); take
exclusive()only for the write phase. - Serve read-only scratch queries on separate DuckDB connections to the same file so reads do not queue behind writes.
- Report
queue_msandexec_msseparately in tool metadata and traces.
Acceptance check
- Test: while a materialize from a deliberately slow source is running, a
SELECT 1on scratch from another sub-agent returns in under 1 s. - Two concurrent materializations from different sources overlap their read phases.
Related: #1479 (Mongo byte-bounded reads fetched one document per round trip; fix in #1480).
- Ngôn ngữ chính
- TypeScript
- Star
- 810
- Fork
- 121
- Merge trung bình
- 1 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 89
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của AltimateAI/altimate-code
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
AltimateAI/altimate-code#1454 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
test: MCP tests fail when the developer's ~/.claude.json has MCP servers (HOME is not sandboxed)Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
AltimateAI/altimate-code#1386 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
AltimateAI/altimate-code#1384 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
AltimateAI/altimate-code#1323 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
AltimateAI/altimate-code#1288 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của AltimateAI/altimate-code
Issue tương tự
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 74/100
components-web-app/docs#260 ·
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
521xueweihan/HelloGitHub#4025 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
ctrf-io/playwright-ctrf-json-reporter#117 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
voidzero-dev/void#478 ·
Maintainer thường phản hồi trong vòng 1 ngày