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

    https://github.com/apache/spark/pull/22947#discussion_r231191913
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/misc.scala
 ---
    @@ -66,6 +66,8 @@ case class AssertTrue(child: Expression) extends 
UnaryExpression with ImplicitCa
     
       override def nullable: Boolean = true
     
    +  override lazy val deterministic: Boolean = false
    --- End diff --
    
    I agree, thanks you all the comments. I'll try and add a new flag. I'll 
update the PR as soon as I have a solution for that. Probably it will take some 
time, though, as I will have to recheck all the usage of `determistic`.... 
Thanks.


---

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

Reply via email to