Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20036#discussion_r158430481
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
---
@@ -118,9 +118,8 @@ case class Like(left: Expression, right: Expression)
extends StringRegexExpressi
if (rVal != null) {
val regexStr =
StringEscapeUtils.escapeJava(escape(rVal.asInstanceOf[UTF8String].toString()))
- // inline mutable state since not many Like operations in a task
val pattern = ctx.addMutableState(patternClass, "patternLike",
- v => s"""$v = ${patternClass}.compile("$regexStr");""",
forceInline = true)
+ v => s"""$v = ${patternClass}.compile("$regexStr");""")
--- End diff --
nit: we can remove `{` and `}` around `patternClass`.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
