Improve error message when non-SELECT statement is used.

cube2222 · cube2222 · commit 1acc4eb58add · 2022-09-25T13:18:48.000+02:00
diff --git a/cmd/root.go b/cmd/root.go
@@ -229,7 +229,11 @@ octosql "SELECT * FROM plugins.plugins"`,
 		if err != nil {
 			return fmt.Errorf("couldn't parse query: %w", err)
 		}
-		logicalPlan, outputOptions, err := parser.ParseNode(statement.(sqlparser.SelectStatement))
+		selectStmt, ok := statement.(sqlparser.SelectStatement)
+		if !ok {
+			return fmt.Errorf("only SELECT statements are supported")
+		}
+		logicalPlan, outputOptions, err := parser.ParseNode(selectStmt)
 		if err != nil {
 			return fmt.Errorf("couldn't parse query: %w", err)
 		}

Original file line number	Diff line number	Diff line change
@@ -229,7 +229,11 @@ octosql "SELECT * FROM plugins.plugins"`,
`229`	`229`	`if err != nil {`
`230`	`230`	`return fmt.Errorf("couldn't parse query: %w", err)`
`231`	`231`	`}`
`232`		`- logicalPlan, outputOptions, err := parser.ParseNode(statement.(sqlparser.SelectStatement))`
	`232`	`+ selectStmt, ok := statement.(sqlparser.SelectStatement)`
	`233`	`+ if !ok {`
	`234`	`+ return fmt.Errorf("only SELECT statements are supported")`
	`235`	`+ }`
	`236`	`+ logicalPlan, outputOptions, err := parser.ParseNode(selectStmt)`
`233`	`237`	`if err != nil {`
`234`	`238`	`return fmt.Errorf("couldn't parse query: %w", err)`
`235`	`239`	`}`