aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorYucai Yu <yucai.yu@intel.com>2016-09-22 17:22:56 -0700
committerReynold Xin <rxin@databricks.com>2016-09-22 17:22:56 -0700
commit79159a1e87f19fb08a36857fc30b600ee7fdc52b (patch)
treed836f0137e46a19b823fc85399e600a07343ae74
parenta1661968310de35e710e3b6784f63a77c44453fc (diff)
downloadspark-79159a1e87f19fb08a36857fc30b600ee7fdc52b.tar.gz
spark-79159a1e87f19fb08a36857fc30b600ee7fdc52b.tar.bz2
spark-79159a1e87f19fb08a36857fc30b600ee7fdc52b.zip
[SPARK-17635][SQL] Remove hardcode "agg_plan" in HashAggregateExec
## What changes were proposed in this pull request? "agg_plan" are hardcoded in HashAggregateExec, which have potential issue, so removing them. ## How was this patch tested? existing tests. Author: Yucai Yu <yucai.yu@intel.com> Closes #15199 from yucai/agg_plan.
-rw-r--r--sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala2
1 files changed, 1 insertions, 1 deletions
diff --git a/sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala b/sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala
index 59e132dfb2..06199ef3e8 100644
--- a/sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala
+++ b/sql/core/src/main/scala/org/apache/spark/sql/execution/aggregate/HashAggregateExec.scala
@@ -552,7 +552,7 @@ case class HashAggregateExec(
} else {
ctx.addMutableState(fastHashMapClassName, fastHashMapTerm,
s"$fastHashMapTerm = new $fastHashMapClassName(" +
- s"agg_plan.getTaskMemoryManager(), agg_plan.getEmptyAggregationBuffer());")
+ s"$thisPlan.getTaskMemoryManager(), $thisPlan.getEmptyAggregationBuffer());")
ctx.addMutableState(
"org.apache.spark.unsafe.KVIterator",
iterTermForFastHashMap, "")