Skip to content

[SPARK-9950][SQL] Wrong Analysis Error for grouping/aggregating on struct fields #8222

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
cloud-fan wants to merge 2 commits into from
Closed

[SPARK-9950][SQL] Wrong Analysis Error for grouping/aggregating on struct fields #8222

cloud-fan wants to merge 2 commits into from

Conversation

@cloud-fan
Copy link
Contributor

@cloud-fan cloud-fan commented Aug 15, 2015

This issue has been fixed by #8215, this PR added regression test for it.

@cloud-fan
Copy link
Contributor Author

cloud-fan commented Aug 15, 2015

cc @marmbrus

@SparkQA
Copy link

SparkQA commented Aug 15, 2015

Test build #40967 has finished for PR 8222 at commit 7e2d8d9.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class FilterNode(condition: Expression, child: LocalNode) extends UnaryLocalNode
    • abstract class LocalNode extends TreeNode[LocalNode]
    • abstract class LeafLocalNode extends LocalNode
    • abstract class UnaryLocalNode extends LocalNode
    • case class ProjectNode(projectList: Seq[NamedExpression], child: LocalNode) extends UnaryLocalNode
    • case class SeqScanNode(output: Seq[Attribute], data: Seq[InternalRow]) extends LeafLocalNode
    • public final class UTF8String implements Comparable<UTF8String>, Externalizable

@SparkQA
Copy link

SparkQA commented Aug 16, 2015

Test build #40980 has finished for PR 8222 at commit 0bbfb1c.

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

asfgit pushed a commit that referenced this pull request Aug 17, 2015
@cloud-fan @marmbrus
[SPARK-9950] [SQL] Wrong Analysis Error for grouping/aggregating on s…
76390ec 
…truct fields

This issue has been fixed by #8215, this PR added regression test for it.

Author: Wenchen Fan <cloud0fan@outlook.com>

Closes #8222 from cloud-fan/minor and squashes the following commits:

0bbfb1c [Wenchen Fan] fix style...
7e2d8d9 [Wenchen Fan] add test

(cherry picked from commit a4acdab)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in a4acdab Aug 17, 2015
@marmbrus
Copy link
Contributor

marmbrus commented Aug 17, 2015

Thanks! Merged to master and 1.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cloud-fan @SparkQA @marmbrus