8000 docs(spec): query memory accounting and optional limit, QRY-012 (#549) by adityamukho · Pull Request #674 · project-minigraf/minigraf · GitHub
[go: up one dir, main page]

Skip to content

docs(spec): query memory accounting and optional limit, QRY-012 (#549) - #674

Merged
adityamukho merged 2 commits into
v3from
docs/549-memory-spec
Oct 21, 2026
Merged

adityamukho merged 2 commits into
v3from
docs/549-memory-spec

Conversation

@adityamukho
@adityamukho adityamukho comme 8000 nted Oct 21, 2026
Copy link
Copy Markdown
Collaborator

Design for #549 (part 7 of 9 of #473): docs/superpowers/specs/2026-10-21-query-memory-limit-design.md.

Decisions settled 2026-10-21:

  • Per-entry byte estimates in every stateful operator, IdSpace provisional ids and rule Relations, one shared meter per query; growth through try_reserve.
  • QRY-012 for both the configured limit and an allocator refusal.
  • OpenOptions::query_memory_limit, off by default (usize::MAX); engine queries only (the executor fallback goes away in streaming engine 9/9: remove the old executor, measure, docs (#473) #551).
  • Keyless (cartesian) hash joins build on the side with the smaller product estimate.
  • Nightly fuzz failure (main) #575 acceptance drains a cursor: :find is a bag, so there is no distinct set to bound.

Part of #473 and #549 (no closing keywords).

🤖 Generated with Claude Code

https://claude.ai/code/session_019kY61Y11gcRD91Y6b7jbGT

adityamukho and others added 2 commits October 21, 2026 09:46
…es, keyless build side (#549)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019kY61Y11gcRD91Y6b7jbGT
@codecov
codecov Bot commented Oct 21, 2026
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@adityamukho
adityamukho merged commit bd28ec7 into v3 Oct 21, 2026
24 of 25 checks passed
@adityamukho
adityamukho deleted the docs/549-memory-spec branch October 21, 2026 04:44
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

330A
0