doc: better error message for VirtualTableFromRemap

vbarua · vbarua · commit ff2e94df95d9 · 2025-09-25T15:25:46.000-07:00
diff --git a/plan/builders.go b/plan/builders.go
@@ -569,7 +569,7 @@ func (b *builder) VirtualTableFromExpr(fieldNames []string, values ...expr.Virtu
 
 func (b *builder) VirtualTableFromExprRemap(fieldNames []string, remap []int32, values ...expr.VirtualTableExpressionValue) (*VirtualTableReadRel, error) {
 	if len(values) == 0 {
-		return nil, fmt.Errorf("%w: must provide at least one set of values for virtual table", substraitgo.ErrInvalidRel)
+		return nil, fmt.Errorf("%w: must provide at least one set of values. Consider EmptyVirtualTable to construct rowless Virtual Table", substraitgo.ErrInvalidArg)
 	}
 
 	nfields := len(fieldNames)

Original file line number	Diff line number	Diff line change
`@@ -569,7 +569,7 @@ func (b *builder) VirtualTableFromExpr(fieldNames []string, values ...expr.Virtu`
`569`	`569`
`570`	`570`	`func (b builder) VirtualTableFromExprRemap(fieldNames []string, remap []int32, values ...expr.VirtualTableExpressionValue) (VirtualTableReadRel, error) {`
`571`	`571`	`if len(values) == 0 {`
`572`		`- return nil, fmt.Errorf("%w: must provide at least one set of values for virtual table", substraitgo.ErrInvalidRel)`
	`572`	`+ return nil, fmt.Errorf("%w: must provide at least one set of values. Consider EmptyVirtualTable to construct rowless Virtual Table", substraitgo.ErrInvalidArg)`
`573`	`573`	`}`
`574`	`574`
`575`	`575`	`nfields := len(fieldNames)`