Skip to content

Clarify DuckLake metadata query intent#19

Open
bill-ph wants to merge 3 commits into
v1.5-variegatafrom
codex/metadata-api-semantics
Open

Clarify DuckLake metadata query intent#19
bill-ph wants to merge 3 commits into
v1.5-variegatafrom
codex/metadata-api-semantics

Conversation

@bill-ph
Copy link
Copy Markdown
Collaborator

@bill-ph bill-ph commented May 19, 2026

Summary

  • No actual behavior change, just some renaming as preparation for real changes
  • split DuckLake metadata execution into Execute, SnapshotQuery, CurrentQuery, and RawQuery
  • classify existing metadata call sites without changing transaction behavior
  • add CI-backed metadata Query intent audit, including comments describing purpose, context, and retirement conditions

Testing

  • cmake --build build/release --target ducklake_extension -j 8
  • ./build/release/test/unittest test/sql/data_inlining/data_inlining_time_travel_reattach.test
  • ./build/release/test/unittest test/sql/data_inlining/data_inlining_flush_schema.test
  • ./build/release/test/unittest test/sql/metadata/ducklake_settings.test
  • python3 scripts/check_metadata_query_intent.py
  • PYTHONPYCACHEPREFIX=/private/tmp/ducklake-pycache python3 -m py_compile scripts/check_metadata_query_intent.py
  • git diff --check

Notes: sqlite/postgres-specific metadata tests were invoked earlier but skipped because the local build is missing sqlite_scanner/postgres_scanner requirements.

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