Skip to content

Issue #17621: Streaming Window Reset#17649

Merged
Mytherin merged 3 commits intoduckdb:v1.3-ossivalisfrom
hawkfish:streaming-reset
May 26, 2025
Merged

Issue #17621: Streaming Window Reset#17649
Mytherin merged 3 commits intoduckdb:v1.3-ossivalisfrom
hawkfish:streaming-reset

Conversation

@hawkfish
Copy link
Contributor

  • Call Reset before evaluation
  • Remove sort to detect hash scan order changes.
  • Remove output because repro query is non-deterministic.

fixes: #17621
fixes: duckdblabs/duckdb-internal#4955

hawkfish added 3 commits May 23, 2025 10:59
* Remove sort to detect hash scan order changes.

fixes: duckdb#17621
fixes: duckdblabs/duckdb-internal#4955
* Remove output because the query is non-deterministic.

fixes: duckdb#17621
fixes: duckdblabs/duckdb-internal#4955
Copy link
Collaborator

@lnkuiper lnkuiper left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@Mytherin Mytherin merged commit 66c2f1e into duckdb:v1.3-ossivalis May 26, 2025
50 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

@hawkfish hawkfish deleted the streaming-reset branch May 26, 2025 17:06
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request Jun 2, 2025
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