From 9ff2e816020ea08214fcb4a9474641a0f1673efb Mon Sep 17 00:00:00 2001 From: yan ma Date: Mon, 15 Jul 2024 20:06:24 +0800 Subject: [PATCH] Revert "[VL] Enable Spark repeat function (#6419)" This reverts commit 62d55cf2bbd3c1b237d773102cd5b1b23c4c600c. --- .../gluten/execution/ScalarFunctionsValidateSuite.scala | 6 ------ cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc | 1 + docs/velox-backend-support-progress.md | 2 +- 3 files changed, 2 insertions(+), 7 deletions(-) diff --git a/backends-velox/src/test/scala/org/apache/gluten/execution/ScalarFunctionsValidateSuite.scala b/backends-velox/src/test/scala/org/apache/gluten/execution/ScalarFunctionsValidateSuite.scala index 39c1b4560646b..adb8be51d66ec 100644 --- a/backends-velox/src/test/scala/org/apache/gluten/execution/ScalarFunctionsValidateSuite.scala +++ b/backends-velox/src/test/scala/org/apache/gluten/execution/ScalarFunctionsValidateSuite.scala @@ -1270,10 +1270,4 @@ class ScalarFunctionsValidateSuite extends FunctionsValidateTest { } } } - - test("repeat") { - runQueryAndCompare("select repeat(c_comment, 5) from customer limit 50") { - checkGlutenOperatorMatch[ProjectExecTransformer] - } - } } diff --git a/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc b/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc index 6f0b9fadec210..8e6dd7c1c31f3 100644 --- a/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc +++ b/cpp/velox/substrait/SubstraitToVeloxPlanValidator.cc @@ -65,6 +65,7 @@ static const std::unordered_set kBlackList = { "concat_ws", "from_json", "json_array_length", + "repeat", "trunc", "sequence", "approx_percentile", diff --git a/docs/velox-backend-support-progress.md b/docs/velox-backend-support-progress.md index 01c730a891f9f..8429759fa381b 100644 --- a/docs/velox-backend-support-progress.md +++ b/docs/velox-backend-support-progress.md @@ -171,7 +171,7 @@ Gluten supports 199 functions. (Drag to right to see all data types) | parse_url | | | | | | | | | | | | | | | | | | | | | | | | position | strpos | | | | | | | | | | | | | | | | | | | | | | | printf | | | | | | | | | | | | | | | | | | | | | | | -| repeat | | repeat | S | | | | | | | | | | | | | | | | | | | | +| repeat | | | | | | | | | | | | | | | | | | | | | | | | replace | replace | replace | S | | | | | | | | | | | S | | | | | | | | | | reverse | reverse | | S | | | | | | | | | | | S | | | | | | | | | | right | | | | | | | | | | | | | | | | | | | | | | |