Github user ioana-delaney commented on a diff in the pull request:

    https://github.com/apache/spark/pull/17546#discussion_r110318802
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/CostBasedJoinReorder.scala
 ---
    @@ -134,7 +132,7 @@ case class CostBasedJoinReorder(conf: SQLConf) extends 
Rule[LogicalPlan] with Pr
      * For cost evaluation, since physical costs for operators are not 
available currently, we use
      * cardinalities and sizes to compute costs.
      */
    -object JoinReorderDP extends PredicateHelper with Logging {
    +case class JoinReorderDP(conf: SQLConf) extends PredicateHelper with 
Logging {
    --- End diff --
    
    @gatorsmile I would like to control the filters on top of the join 
enumeration. We might have other filters, e.g. left-deep trees only.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to