Skip to content

Conversation

@marmbrus
Copy link
Contributor

We need to convert the case classes into Rows.

@SparkQA
Copy link

SparkQA commented Aug 26, 2014

QA tests have started for PR 2133 at commit d8d0b76.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 26, 2014

QA tests have finished for PR 2133 at commit d8d0b76.

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

@SparkQA
Copy link

SparkQA commented Aug 27, 2014

QA tests have started for PR 2133 at commit 8e29b1c.

  • This patch does not merge cleanly!

Conflicts:
	sql/core/src/main/scala/org/apache/spark/sql/execution/basicOperators.scala
@SparkQA
Copy link

SparkQA commented Aug 27, 2014

QA tests have finished for PR 2133 at commit 8e29b1c.

  • This patch passes unit tests.
  • This patch does not merge cleanly!

asfgit pushed a commit that referenced this pull request Aug 28, 2014
We need to convert the case classes into Rows.

Author: Michael Armbrust <[email protected]>

Closes #2133 from marmbrus/structUdfs and squashes the following commits:

189722f [Michael Armbrust] Merge remote-tracking branch 'origin/master' into structUdfs
8e29b1c [Michael Armbrust] Use existing function
d8d0b76 [Michael Armbrust] Fix udfs that return structs

(cherry picked from commit 76e3ba4)
Signed-off-by: Michael Armbrust <[email protected]>
@asfgit asfgit closed this in 76e3ba4 Aug 28, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
We need to convert the case classes into Rows.

Author: Michael Armbrust <[email protected]>

Closes apache#2133 from marmbrus/structUdfs and squashes the following commits:

189722f [Michael Armbrust] Merge remote-tracking branch 'origin/master' into structUdfs
8e29b1c [Michael Armbrust] Use existing function
d8d0b76 [Michael Armbrust] Fix udfs that return structs
@marmbrus marmbrus deleted the structUdfs branch September 22, 2014 19:54
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.

2 participants