Skip to content

Conversation

@aray
Copy link
Contributor

@aray aray commented Dec 8, 2015

Delays application of ResolvePivot until all aggregates are resolved to prevent problems with UnresolvedFunction and adds unit test

…to prevent problems with UnresolvedFunction and unit test
@SparkQA
Copy link

SparkQA commented Dec 8, 2015

Test build #47343 has finished for PR 10202 at commit 8a3f555.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@aray
Copy link
Contributor Author

aray commented Dec 8, 2015

@yhuai can you take a look at this small patch to pivot?

@yhuai
Copy link
Contributor

yhuai commented Dec 8, 2015

LGTM. Thank you for the fix! I am merging it to master and branch 1.6.

asfgit pushed a commit that referenced this pull request Dec 8, 2015
…unction

Delays application of ResolvePivot until all aggregates are resolved to prevent problems with UnresolvedFunction and adds unit test

Author: Andrew Ray <ray.andrew@gmail.com>

Closes #10202 from aray/sql-pivot-unresolved-function.

(cherry picked from commit 4bcb894)
Signed-off-by: Yin Huai <yhuai@databricks.com>
@asfgit asfgit closed this in 4bcb894 Dec 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants