Skip to content

[SPARK-56875][SQL] Refactor time/session window resolution code to be reused in the single-pass Analyzer#55895

Closed
vladimirg-db wants to merge 1 commit into
apache:masterfrom
vladimirg-db:vladimir-golubev_data/time-windowing-refactor
Closed

[SPARK-56875][SQL] Refactor time/session window resolution code to be reused in the single-pass Analyzer#55895
vladimirg-db wants to merge 1 commit into
apache:masterfrom
vladimirg-db:vladimir-golubev_data/time-windowing-refactor

Conversation

@vladimirg-db
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Refactor time/session window resolution code to be reused in the single-pass Analyzer.

Why are the changes needed?

Unblocks single-pass Analyzer support for time/sessnon windowing.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Existing/new tests.

Was this patch authored or co-authored using generative AI tooling?

Claude.

@cloud-fan
Copy link
Copy Markdown
Contributor

thanks, merging to master/4.x!

@cloud-fan cloud-fan closed this in 7569ba6 May 15, 2026
cloud-fan pushed a commit that referenced this pull request May 15, 2026
… reused in the single-pass Analyzer

### What changes were proposed in this pull request?

Refactor time/session window resolution code to be reused in the single-pass Analyzer.

### Why are the changes needed?

Unblocks single-pass Analyzer support for time/sessnon windowing.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Existing/new tests.

### Was this patch authored or co-authored using generative AI tooling?

Claude.

Closes #55895 from vladimirg-db/vladimir-golubev_data/time-windowing-refactor.

Authored-by: Vladimir Golubev <vladimir.golubev@databricks.com>
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
(cherry picked from commit 7569ba6)
Signed-off-by: Wenchen Fan <wenchen@databricks.com>
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