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

    https://github.com/apache/spark/pull/20407#discussion_r167351883
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala ---
    @@ -262,6 +262,10 @@ abstract class SparkStrategies extends 
QueryPlanner[SparkPlan] {
             joins.CartesianProductExec(planLater(left), planLater(right), 
condition) :: Nil
     
           case logical.Join(left, right, joinType, condition) =>
    +        if (!SQLConf.get.allowNestedJoinFallback) {
    +          throw new AnalysisException("The only JOIN strategy available 
for this plan is " +
    +            s"BroadcastNestedLoopJoin, but 
`${SQLConf.ALLOW_NESTEDJOIN_FALLBACK}` is `false`.")
    --- End diff --
    
    nit: `SQLConf.ALLOW_NESTEDJOIN_FALLBACK.key`


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to