You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by liufengdb <gi...@git.apache.org> on 2018/02/09 21:34:57 UTC

[GitHub] spark pull request #20407: [SPARK-23124][SQL] Allow to disable BroadcastNest...

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-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org