Merge pull request #2977 from dolthub/zachmu/offset

zachmu · web-flow · commit e116aa682aaf · 2025-05-14T14:33:18.000-07:00
Bug fix for offset without limit
diff --git a/sql/planbuilder/select.go b/sql/planbuilder/select.go
@@ -127,7 +127,7 @@ func (b *Builder) buildSelect(inScope *scope, s *ast.Select) (outScope *scope) {
 }
 
 func (b *Builder) buildLimit(inScope *scope, limit *ast.Limit) sql.Expression {
-	if limit != nil {
+	if limit != nil && limit.Rowcount != nil {
 		return b.buildLimitVal(inScope, limit.Rowcount)
 	}
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,7 @@ func (b Builder) buildSelect(inScope scope, s ast.Select) (outScope scope) {`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`func (b Builder) buildLimit(inScope scope, limit *ast.Limit) sql.Expression {`
`130`		`- if limit != nil {`
	`130`	`+ if limit != nil && limit.Rowcount != nil {`
`131`	`131`	`return b.buildLimitVal(inScope, limit.Rowcount)`
`132`	`132`	`}`
`133`	`133`	`return nil`