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