Merge pull request #2887 from dolthub/nicktobey/json-length

Correct the behavior of passing a JSON object into the JSON_LENGTH function.

Author: nicktobey

sql/expression/function/json/json_length.go

@@ -117,7 +117,7 @@ func (j *JsonLength) Eval(ctx *sql.Context, row sql.Row) (interface{}, error) {
 			return nil, nil
 		}
 		return len(v), nil
-	case map[any]any:
+	case map[string]interface{}:
 		return len(v), nil
 	default:
 		return 1, nil

sql/expression/function/json/jsontests/json_length_test.go

@@ -74,7 +74,11 @@ func TestJsonLength(t *testing.T) {
 			row: sql.Row{`{"a": 1}`},
 			exp: 1,
 		},
-
+		{
+			f:   f1,
+			row: sql.Row{`{"a": 1, "b": 2}`},
+			exp: 2,
+		},
 		{
 			f:   f2,
 			row: sql.Row{`{"a": [1, false]}`, nil},