Skip to content

Defer Bloom Filter Pushdown until it's done - #22218

Merged
Mytherin merged 1 commit into
duckdb:v1.5-variegatafrom
lnkuiper:defer_bf
Apr 26, 2026
Merged

Mytherin merged 1 commit into
duckdb:v1.5-variegatafrom
lnkuiper:defer_bf

Conversation

@lnkuiper

Copy link
Copy Markdown
Member

Fixes #22075
Fixes https://github.com/duckdblabs/duckdb-internal/issues/8895

Shouldn't be possible to probe it before it has been built, so this PR defers pushing it down until it has been built.

@lnkuiper

lnkuiper commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

test/sql/join/inner/test_join_is_not_distinct.test times out - not sure if related. I'll have a look tomorrow.

EDIT: cannot get it to time out or fail locally even when trying different numbers of threads, going to assume a CI hiccup and rerun.

@Mytherin
Mytherin merged commit 5909259 into duckdb:v1.5-variegata Apr 26, 2026
72 of 73 checks passed
@Mytherin

Copy link
Copy Markdown
Collaborator

Thanks!

@lnkuiper
lnkuiper deleted the defer_bf branch May 12, 2026 13:55
kaka11chen pushed a commit to kaka11chen/astrovela_vane that referenced this pull request Aug 14, 2026
Fixes duckdb/duckdb#22075
Fixes https://github.com/duckdblabs/duckdb-internal/issues/8895

Shouldn't be possible to probe it before it has been built, so this PR
defers pushing it down until it has been built.

Upstream-Commit: duckdb/duckdb@5909259
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.

2 participants