Checking for StringType implementations now that Doltgres extended ty…

…pes can implement query types without implementing StringType
dolthub · Oct 7, 2024 · 1d28860 · 1d28860
1 parent fdd2454
commit 1d28860
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/go/libraries/doltcore/schema/schema_impl.go b/go/libraries/doltcore/schema/schema_impl.go
@@ -524,9 +524,10 @@ func (si *schemaImpl) getKeyColumnsDescriptor(convertAddressColumns bool) val.Tu
 			}
 		}
 		tt = append(tt, t)
-		if queryType == query.Type_CHAR || queryType == query.Type_VARCHAR || queryType == query.Type_TEXT {
+		stringType, isStringType := sqlType.(sql.StringType)
+		if isStringType && (queryType == query.Type_CHAR || queryType == query.Type_VARCHAR || queryType == query.Type_TEXT) {
 			useCollations = true
-			collations = append(collations, sqlType.(sql.StringType).Collation())
+			collations = append(collations, stringType.Collation())
 		} else {
 			collations = append(collations, sql.Collation_Unspecified)
 		}