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

    https://github.com/apache/spark/pull/19952#discussion_r156550872
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
 ---
    @@ -574,51 +539,90 @@ case class FilterEstimation(plan: Filter) extends 
Logging {
       }
     
       /**
    -   * Returns the selectivity percentage for binary condition in the 
column's
    -   * current valid range [min, max]
    -   *
    -   * @param op a binary comparison operator
    -   * @param histogram a numeric equi-height histogram
    -   * @param max the upper bound of the current valid range for a given 
column
    -   * @param min the lower bound of the current valid range for a given 
column
    -   * @param datumNumber the numeric value of a literal
    -   * @return the selectivity percentage for a condition in the current 
range.
    +   * Computes the possibility of a equal predicate using histogram.
    --- End diff --
    
    nit: an equality predicate


---

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

Reply via email to