Enable json_array_length

WangGuangxin · WangGuangxin · commit cb554f8234f6 · 2025-03-07T19:04:07.000+08:00
diff --git a/backends-velox/src/test/scala/org/apache/gluten/execution/JsonFunctionsValidateSuite.scala b/backends-velox/src/test/scala/org/apache/gluten/execution/JsonFunctionsValidateSuite.scala
@@ -47,7 +47,7 @@ class JsonFunctionsValidateSuite extends FunctionsValidateSuite {
     }
   }
 
-  ignore("json_array_length") {
+  test("json_array_length") {
     runQueryAndCompare(
       s"select *, json_array_length(string_field1) " +
         s"from datatab limit 5")(checkGlutenOperatorMatch[ProjectExecTransformer])
diff --git a/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc b/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc
@@ -61,7 +61,6 @@ const std::unordered_set<std::string> kRegexFunctions = {
 const std::unordered_set<std::string> kBlackList = {
     "split_part",
     "factorial",
-    "json_array_length",
     "trunc",
     "sequence",
     "approx_percentile",
diff --git a/docs/velox-backend-scalar-function-support.md b/docs/velox-backend-scalar-function-support.md
@@ -179,7 +179,7 @@
 |-------------------|---------------------|----------|----------------|
 | from_json         | JsonToStructs       |          |                |
 | get_json_object   | GetJsonObject       | S        |                |
-| json_array_length | LengthOfJsonArray   |          |                |
+| json_array_length | LengthOfJsonArray   | S        |                |
 | json_object_keys  | JsonObjectKeys      |          |                |
 | json_tuple        | JsonTuple           | S        |                |
 | schema_of_json    | SchemaOfJson        |          |                |

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ class JsonFunctionsValidateSuite extends FunctionsValidateSuite {`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`
`50`		`- ignore("json_array_length") {`
	`50`	`+ test("json_array_length") {`
`51`	`51`	`runQueryAndCompare(`
`52`	`52`	`s"select *, json_array_length(string_field1) " +`
`53`	`53`	`s"from datatab limit 5")(checkGlutenOperatorMatch[ProjectExecTransformer])`