Skip to content

Commit

Permalink
Revert "[VL] Add a test to validate substring_index function (apache#…
Browse files Browse the repository at this point in the history
…6393)"

This reverts commit 997c6e3.
  • Loading branch information
yma11 committed Jul 15, 2024
1 parent 7a951b3 commit 8bf50ca
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 36 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1235,39 +1235,4 @@ class ScalarFunctionsValidateSuite extends FunctionsValidateTest {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}

test("Test substring_index") {
withTempView("substring_index_table") {
withTempPath {
path =>
Seq[(String, String, Int)](
("www.apache.org", ".", 3),
("www.apache.org", ".", 2),
("www.apache.org", ".", 1),
("www.apache.org", ".", 0),
("www.apache.org", ".", -1),
("www.apache.org", ".", -2),
("www.apache.org", ".", -3),
("www.apache.org", "", 1),
("www.apache.org", "#", 1),
("www||apache||org", "||", 2),
("www||apache||org", "||", -2),
("", ".", 1),
("||||||", "|||", 3),
("||||||", "|||", -4)
)
.toDF("str", "delim", "count")
.write
.parquet(path.getCanonicalPath)
spark.read.parquet(path.getCanonicalPath).createOrReplaceTempView("substring_index_table")
runQueryAndCompare(
"""
|select substring_index(str, delim, count) from substring_index_table
|""".stripMargin
) {
checkGlutenOperatorMatch[ProjectExecTransformer]
}
}
}
}
}
2 changes: 1 addition & 1 deletion docs/velox-backend-support-progress.md
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ Gluten supports 199 functions. (Drag to right to see all data types)
| startswith | | startsWith | | | | | | | | | | | | | | | | | | | | |
| substr, substring | substr | substring | S | | | | | | | | | | | S | | | | | | | | |
| substring, substr | substr | substring | S | | | | | | | | | | | S | | | | | | | | |
| substring_index | | substring_index | S | | | | | | | | | | | | | | | | | | | |
| substring_index | | | | | | | | | | | | | | | | | | | | | | |
| translate | | | | | | | | | | | | | | | | | | | | | | |
| trim | trim | trim | S | | | | | | | | | | | S | | | | | | | | |
| ucase, upper | upper | upper | S | | | | | | | | | | | S | | | | | | | | |
Expand Down

0 comments on commit 8bf50ca

Please sign in to comment.