You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by hv...@apache.org on 2016/12/07 12:44:19 UTC
spark git commit: [SPARK-17760][SQL] AnalysisException with dataframe
pivot when groupBy column is not attribute
Repository: spark
Updated Branches:
refs/heads/master c496d03b5 -> f1fca81b1
[SPARK-17760][SQL] AnalysisException with dataframe pivot when groupBy column is not attribute
## What changes were proposed in this pull request?
Fixes AnalysisException for pivot queries that have group by columns that are expressions and not attributes by substituting the expressions output attribute in the second aggregation and final projection.
## How was this patch tested?
existing and additional unit tests
Author: Andrew Ray <ra...@gmail.com>
Closes #16177 from aray/SPARK-17760.
Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/f1fca81b
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/f1fca81b
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/f1fca81b
Branch: refs/heads/master
Commit: f1fca81b165c5a673f7d86b268e04ea42a6c267e
Parents: c496d03
Author: Andrew Ray <ra...@gmail.com>
Authored: Wed Dec 7 04:44:14 2016 -0800
Committer: Herman van Hovell <hv...@databricks.com>
Committed: Wed Dec 7 04:44:14 2016 -0800
----------------------------------------------------------------------
.../org/apache/spark/sql/catalyst/analysis/Analyzer.scala | 5 +++--
.../scala/org/apache/spark/sql/DataFramePivotSuite.scala | 8 ++++++++
2 files changed, 11 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/spark/blob/f1fca81b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
----------------------------------------------------------------------
diff --git a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
index ed6e17a..58f98d5 100644
--- a/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
+++ b/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
@@ -463,14 +463,15 @@ class Analyzer(
.toAggregateExpression()
, "__pivot_" + a.sql)()
}
- val secondAgg = Aggregate(groupByExprs, groupByExprs ++ pivotAggs, firstAgg)
+ val groupByExprsAttr = groupByExprs.map(_.toAttribute)
+ val secondAgg = Aggregate(groupByExprsAttr, groupByExprsAttr ++ pivotAggs, firstAgg)
val pivotAggAttribute = pivotAggs.map(_.toAttribute)
val pivotOutputs = pivotValues.zipWithIndex.flatMap { case (value, i) =>
aggregates.zip(pivotAggAttribute).map { case (aggregate, pivotAtt) =>
Alias(ExtractValue(pivotAtt, Literal(i), resolver), outputName(value, aggregate))()
}
}
- Project(groupByExprs ++ pivotOutputs, secondAgg)
+ Project(groupByExprsAttr ++ pivotOutputs, secondAgg)
} else {
val pivotAggregates: Seq[NamedExpression] = pivotValues.flatMap { value =>
def ifExpr(expr: Expression) = {
http://git-wip-us.apache.org/repos/asf/spark/blob/f1fca81b/sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala
----------------------------------------------------------------------
diff --git a/sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala
index 1bbe135..a8d854c 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/DataFramePivotSuite.scala
@@ -208,4 +208,12 @@ class DataFramePivotSuite extends QueryTest with SharedSQLContext{
)
}
+ test("pivot with column definition in groupby") {
+ checkAnswer(
+ courseSales.groupBy(substring(col("course"), 0, 1).as("foo"))
+ .pivot("year", Seq(2012, 2013))
+ .sum("earnings"),
+ Row("d", 15000.0, 48000.0) :: Row("J", 20000.0, 30000.0) :: Nil
+ )
+ }
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org