Skip to content

Commit 0a0c869

Browse files
committed
docs: add getIncompatibleReasons for CometStringTranslate
1 parent f2257a1 commit 0a0c869

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

spark/src/main/scala/org/apache/comet/serde/strings.scala

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -103,10 +103,14 @@ object CometOctetLength extends CometScalarFunction[OctetLength]("octet_length")
103103
}
104104

105105
object CometStringTranslate extends CometScalarFunction[StringTranslate]("translate") {
106+
private val incompatReason =
107+
"DataFusion's translate iterates over Unicode graphemes (Spark uses code points) and" +
108+
" substitutes U+0000 instead of treating it as a deletion sentinel"
109+
110+
override def getIncompatibleReasons(): Seq[String] = Seq(incompatReason)
111+
106112
override def getSupportLevel(expr: StringTranslate): SupportLevel = Incompatible(
107-
Some(
108-
"DataFusion's translate iterates over Unicode graphemes (Spark uses code points) and" +
109-
" substitutes U+0000 instead of treating it as a deletion sentinel"))
113+
Some(incompatReason))
110114
}
111115

112116
object CometInitCap extends CometScalarFunction[InitCap]("initcap") {

0 commit comments

Comments
 (0)