Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19897#discussion_r155222416
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/regexpExpressions.scala
 ---
    @@ -119,33 +119,34 @@ case class Like(left: Expression, right: Expression) 
extends StringRegexExpressi
           if (rVal != null) {
             val regexStr =
               
StringEscapeUtils.escapeJava(escape(rVal.asInstanceOf[UTF8String].toString()))
    -        ctx.addMutableState(patternClass, pattern,
    -          s"""$pattern = ${patternClass}.compile("$regexStr");""")
    --- End diff --
    
    but we are adding a new one every time, so we are compiling it every time. 
Am I wrong?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to