Revert

lib/sql/escape.go

@@ -17,7 +17,19 @@ type NameArgs struct {
 // symbolsToEscape are additional keywords that we need to escape
 var symbolsToEscape = []string{":"}
 
-func needsEscaping(name string, uppercaseEscNames bool, destKind constants.DestinationKind) bool {
+func EscapeNameIfNecessary(name string, uppercaseEscNames bool, args *NameArgs) string {
+	if args == nil || !args.Escape {
+		return name
+	}
+
+	if needsEscaping(name, args.DestKind) {
+		return escapeName(name, uppercaseEscNames, args.DestKind)
+	}
+
+	return name
+}
+
+func needsEscaping(name string, destKind constants.DestinationKind) bool {
 	var reservedKeywords []string
 	if destKind == constants.Redshift {
 		reservedKeywords = constants.RedshiftReservedKeywords
@@ -49,18 +61,6 @@ func needsEscaping(name string, uppercaseEscNames bool, destKind constants.Desti
 	return needsEscaping
 }
 
-func EscapeNameIfNecessary(name string, uppercaseEscNames bool, args *NameArgs) string {
-	if args == nil || !args.Escape {
-		return name
-	}
-
-	if needsEscaping(name, uppercaseEscNames, args.DestKind) {
-		return escapeName(name, uppercaseEscNames, args.DestKind)
-	}
-
-	return name
-}
-
 func escapeName(name string, uppercaseEscNames bool, destKind constants.DestinationKind) string {
 	if uppercaseEscNames {
 		name = strings.ToUpper(name)

lib/sql/escape_test.go

@@ -7,7 +7,7 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-func TestEscapeName(t *testing.T) {
+func TestEscapeNameIfNecessary(t *testing.T) {
 	type _testCase struct {
 		name                     string
 		nameToEscape             string