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

    https://github.com/apache/spark/pull/20670#discussion_r171463022
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/QueryPlanConstraints.scala
 ---
    @@ -22,21 +22,24 @@ import org.apache.spark.sql.catalyst.expressions._
     
     trait QueryPlanConstraints { self: LogicalPlan =>
     
    +  /**
    +   * An [[ExpressionSet]] that contains an additional set of constraints, 
such as equality
    +   * constraints and `isNotNull` constraints, etc.
    +   */
    +  lazy val allConstraints: ExpressionSet = ExpressionSet(validConstraints
    --- End diff --
    
    We still need `if (conf.constraintPropagationEnabled)`


---

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

Reply via email to