-
- Downloads
[SPARK-20972][SQL] rename HintInfo.isBroadcastable to broadcast
## What changes were proposed in this pull request? `HintInfo.isBroadcastable` is actually not an accurate name, it's used to force the planner to broadcast a plan no matter what the data size is, via the hint mechanism. I think `forceBroadcast` is a better name. And `isBroadcastable` only have 2 possible values: `Some(true)` and `None`, so we can just use boolean type for it. ## How was this patch tested? existing tests. Author: Wenchen Fan <wenchen@databricks.com> Closes #18189 from cloud-fan/stats.
Showing
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/ResolveHints.scala 3 additions, 3 deletions...org/apache/spark/sql/catalyst/analysis/ResolveHints.scala
- sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/hints.scala 7 additions, 9 deletions...a/org/apache/spark/sql/catalyst/plans/logical/hints.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/ResolveHintsSuite.scala 10 additions, 10 deletions...pache/spark/sql/catalyst/analysis/ResolveHintsSuite.scala
- sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/BasicStatsEstimationSuite.scala 3 additions, 3 deletions.../catalyst/statsEstimation/BasicStatsEstimationSuite.scala
- sql/core/src/main/scala/org/apache/spark/sql/execution/SparkStrategies.scala 1 addition, 1 deletion...cala/org/apache/spark/sql/execution/SparkStrategies.scala
- sql/core/src/main/scala/org/apache/spark/sql/functions.scala 1 addition, 1 deletionsql/core/src/main/scala/org/apache/spark/sql/functions.scala
Loading
Please register or sign in to comment