Skip to content

Commit de05d7e

Browse files
authored
Merge pull request #113 from stackql/claude/fix-window-function-execution-01WbDt4A6kMPzL5KJmLWAHBU
fix: preserve OVER clause during window function rewriting
2 parents 5fd98aa + 3874d91 commit de05d7e

File tree

7 files changed

+756
-745
lines changed

7 files changed

+756
-745
lines changed

internal/stackql-parser-fork/go/vt/sqlparser/ast.go

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1987,7 +1987,13 @@ func (node *WindowSpec) Format(buf *TrackedBuffer) {
19871987
if needsSpace {
19881988
buf.WriteString(" ")
19891989
}
1990-
buf.astPrintf(node, "order by %v", node.OrderBy)
1990+
// Don't use %v for OrderBy as OrderBy.Format adds its own "order by" prefix
1991+
buf.WriteString("order by ")
1992+
prefix := ""
1993+
for _, n := range node.OrderBy {
1994+
buf.astPrintf(node, "%s%v", prefix, n)
1995+
prefix = ", "
1996+
}
19911997
needsSpace = true
19921998
}
19931999
if node.Frame != nil {

0 commit comments

Comments
 (0)