Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20495#discussion_r166205775
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/stringExpressions.scala
---
@@ -1655,15 +1655,17 @@ case class Left(str: Expression, len: Expression,
child: Expression) extends Run
*/
// scalastyle:off line.size.limit
@ExpressionDescription(
- usage = "_FUNC_(expr) - Returns the character length of `expr` or number
of bytes in binary data.",
+ usage = "_FUNC_(expr) - Returns the character length of `expr` or number
of bytes in binary data. " +
--- End diff --
We should be consistent, either `character string` vs `binary string`, or
`string data` vs `binary data`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
