[ga-format-pr] Run ./format_repo.sh to fix formatting

Author: fulghum

enginetest/queries/query_plans.go

@@ -2,6 +2,7 @@ package plan
 
 import (
 	"github.com/dolthub/go-mysql-server/sql"
+	"github.com/dolthub/go-mysql-server/sql/expression"
 	"github.com/dolthub/go-mysql-server/sql/transform"
 )
 
@@ -20,6 +21,14 @@ func AliasSubqueryString(e sql.Expression) string {
 	if err != nil {
 		panic(err)
 	}
+
+	// String literal values are quoted when their String() method is called, so to avoid that, we
+	// check if we're dealing with a string literal and use it's raw value if so.
+	if literal, ok := e.(*expression.Literal); ok {
+		if s, ok := literal.Value().(string); ok {
+			return s
+		}
+	}
 	return e.String()
 }
 

sql/plan/str_expr.go

@@ -242,6 +242,9 @@ func selectExprNeedsAlias(e *ast.AliasedExpr, expr sql.Expression) bool {
 			return true
 		}
 	})
+	if complex {
+		return true
+	}
 
 	// If the expression's string representation is quoted, trim the quotes before comparing it to the input expression.
 	// InputExpression is assigned in the Vitess layer, and it always trims quotes at that time, too.
@@ -250,7 +253,7 @@ func selectExprNeedsAlias(e *ast.AliasedExpr, expr sql.Expression) bool {
 		exprString = exprString[1 : len(exprString)-1]
 	}
 
-	// If the expression's input value matches expr.String(), then we know that it is referenceable and
-	// does not need an alias.
-	return complex || e.InputExpression != exprString
+	// If the expression's input value does not match expr.String(), then we know that it is not
+	// referenceable and will need an alias.
+	return e.InputExpression != exprString
 }

sql/planbuilder/project.go

@@ -17,8 +17,9 @@ package types
 import (
 	"testing"
 
-	"github.com/dolthub/go-mysql-server/sql"
 	"github.com/stretchr/testify/assert"
+
+	"github.com/dolthub/go-mysql-server/sql"
 )
 
 func TestIsGeometry(t *testing.T) {