Skip to content

Clarify DuckLake metadata query intent#18

Closed
bill-ph wants to merge 2 commits into
PostHog:v1.5-variegatafrom
bill-ph:codex/metadata-api-semantics
Closed

Clarify DuckLake metadata query intent#18
bill-ph wants to merge 2 commits into
PostHog:v1.5-variegatafrom
bill-ph:codex/metadata-api-semantics

Conversation

@bill-ph
Copy link
Copy Markdown
Collaborator

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

Summary

  • 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

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.

@bill-ph
Copy link
Copy Markdown
Collaborator Author

bill-ph commented May 19, 2026

Closing in favor of repo-owned PR #19 from PostHog:codex/metadata-api-semantics.

@bill-ph bill-ph closed this May 19, 2026
@bill-ph bill-ph deleted the codex/metadata-api-semantics branch May 19, 2026 14:10
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