Skip to content

fix(sql): WINDOW JOIN with ORDER BY ts DESC returns error#6624

Merged
bluestreak01 merged 1 commit intomasterfrom
puzpuzpuz_window_join_order_by_desc
Jan 11, 2026
Merged

fix(sql): WINDOW JOIN with ORDER BY ts DESC returns error#6624
bluestreak01 merged 1 commit intomasterfrom
puzpuzpuz_window_join_order_by_desc

Conversation

@puzpuzpuz
Copy link
Copy Markdown
Contributor

Fixes left side of time series join doesn't have ASC timestamp order error when ORDER BY timestamp DESC is used in a WINDOW JOIN query.

@puzpuzpuz puzpuzpuz self-assigned this Jan 10, 2026
@puzpuzpuz puzpuzpuz added Bug Incorrect or unexpected behavior SQL Issues or changes relating to SQL execution labels Jan 10, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 10, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@glasstiger
Copy link
Copy Markdown
Contributor

[PR Coverage check]

😍 pass : 1 / 1 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/griffin/SqlOptimiser.java 1 1 100.00%

@questdb-butler
Copy link
Copy Markdown

⚠️ Enterprise CI Failed

The enterprise test suite failed for this PR.

Build: View Details
Tested Commit: fecc84ca5c07f5d779b10f523b97ab2054e02c83

Please investigate the failure before merging.

@bluestreak01 bluestreak01 merged commit 209f01d into master Jan 11, 2026
40 of 43 checks passed
@bluestreak01 bluestreak01 deleted the puzpuzpuz_window_join_order_by_desc branch January 11, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Incorrect or unexpected behavior SQL Issues or changes relating to SQL execution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants