Skip to content

Update Postgres extension tests#503

Merged
adsharma merged 5 commits into
mainfrom
update-postgres-extension-tests
May 20, 2026
Merged

Update Postgres extension tests#503
adsharma merged 5 commits into
mainfrom
update-postgres-extension-tests

Conversation

@adsharma
Copy link
Copy Markdown
Contributor

@adsharma adsharma commented May 19, 2026

Summary

Earlier commit removed some tests to get pgembed working. This one tries to restore it to the same coverage as before.

@adsharma adsharma marked this pull request as ready for review May 19, 2026 23:55
@adsharma adsharma force-pushed the update-postgres-extension-tests branch 2 times, most recently from eeea3b5 to d9d1209 Compare May 20, 2026 13:38
adsharma added 5 commits May 20, 2026 13:49
Normalize the requested database type before building the missing extension
hint so DuckDB, Postgres, and SQLite attachments all get a consistent
load-extension suggestion.
Only remove predicates that can be converted into table-function column
predicates, leaving unsupported filters above the function call and
covering SHOW_INDEXES filtering in the DDL tests.
Carry relationship filters through the foreign join rewrite and keep graph
properties available until projection pushdown can prune them, avoiding
missing columns after PostgreSQL pushdowns.
Add a pgembed-backed repro script that loads the Postgres extension,
attaches a temporary database, and checks select-star projection and
filter pushdown behavior.
Advance the extension submodule to the restored Postgres test coverage
and SQL_QUERY pushdown fixes that exercise the optimizer behavior
updated in this branch.
@adsharma adsharma force-pushed the update-postgres-extension-tests branch from 13c1f16 to 74a86a3 Compare May 20, 2026 20:57
@adsharma adsharma merged commit c6aa29b into main May 20, 2026
4 checks passed
@adsharma adsharma deleted the update-postgres-extension-tests branch May 20, 2026 21:25
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