Fixup TryFlushCachingOperators to avoid doubly emitted chunks by carlopi · Pull Request #25331 · duckdb/duckdb · GitHub
Skip to content

Fixup TryFlushCachingOperators to avoid doubly emitted chunks - #25331

Merged
lnkuiper merged 1 commit into
duckdb:v1.5-variegatafrom
carlopi:fixes_caching_operators_25306
Sep 4, 2026
Merged

lnkuiper merged 1 commit into
duckdb:v1.5-variegatafrom
carlopi:fixes_caching_operators_25306

Conversation

@carlopi

@carlopi carlopi commented Sep 3, 2026

Copy link
Copy Markdown
Member

Fixes #25306

Proper CI test is "will everything else stay in place"?

@github-actions github-actions Bot added the Member label Sep 3, 2026
@carlopi

carlopi commented Sep 3, 2026

Copy link
Copy Markdown
Member Author

Comment thread test/sql/limit/streaming_limit_pipeline_flush2.test Outdated
@carlopi
carlopi force-pushed the fixes_caching_operators_25306 branch from 600eeb9 to dc0eee1 Compare September 3, 2026 16:12
@carlopi
carlopi marked this pull request as draft September 3, 2026 16:13
@carlopi
carlopi marked this pull request as ready for review September 3, 2026 16:13

@kaka11chen kaka11chen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

query II ?

@carlopi
carlopi force-pushed the fixes_caching_operators_25306 branch from dc0eee1 to d8235e4 Compare September 3, 2026 22:30
@carlopi
carlopi marked this pull request as draft September 3, 2026 22:30
@carlopi
carlopi marked this pull request as ready for review September 3, 2026 22:30

@kaka11chen kaka11chen left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lnkuiper
lnkuiper merged commit 2f10cfb into duckdb:v1.5-variegata Sep 4, 2026
107 checks passed
@carlopi
carlopi deleted the fixes_caching_operators_25306 branch September 4, 2026 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants