Skip to content

spec 023: duckdb-acl after spec 150 stage 4b - #23

Open
VGSML wants to merge 1 commit into
mainfrom
023-pin-duckdb-acl-4b
Open

VGSML wants to merge 1 commit into
mainfrom
023-pin-duckdb-acl-4b

Conversation

@VGSML

@VGSML VGSML commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Spec: specs/023-pin-duckdb-acl-4b/spec.md.

This moves duckdb-acl from 4acccd7 (spec 022) to 7eede0b (its #285, spec 150 stage 4b).

What changes

  • A resource group's limits are read by grammar rules instead of a text reader. Each item is a word, then an optional =, then one value. A value is a number and its unit together (32MiB), a signed number, or a quoted string.
  • The limits function now refuses inputs that used to escape it:
    • a size past int64, which used to raise an internal error that invalidated the database;
    • a JSON integer past int64, which used to wrap;
    • input that made std::stoll throw.

What stays the same

  • MgmtCall::Text behaves as before for this repository's two calls.
  • The usage lines are unchanged.
  • The source list, ext-common and quack do not move.

Tests

Locally:

  • HUGR_NODE_E2E=1 GEN=ninja make: builds.
  • test/sql/*: 14 files, 1043 assertions, all green.
  • make test-cpp: 4/4 green.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LdLs2iBQygBKtsApdH9Sxi

duckdb-acl 4acccd7 -> 7eede0b (#285): a resource group's limits are read by
rules (a word, `=` or not, one value - a number and its unit together, a signed
number, a quoted one); the limits function refuses what escaped it (a size past
int64 invalidated the database, a JSON integer past int64 wrapped, stoll's
exceptions). MgmtCall::Text keeps this repository's two calls as they were.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LdLs2iBQygBKtsApdH9Sxi
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