Skip to content

Conversation

@rxin
Copy link
Contributor

@rxin rxin commented Mar 24, 2016

What changes were proposed in this pull request?

unionAll has been deprecated in SPARK-14088.

How was this patch tested?

Should be covered by all existing tests.

@rxin
Copy link
Contributor Author

rxin commented Mar 24, 2016

cc @davies

* @group typedrel
* @since 2.0.0
*/
* @since 2.0.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

@davies
Copy link
Contributor

davies commented Mar 25, 2016

LGTM, one minor comment

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54117 has finished for PR 11946 at commit 922ec43.

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

@SparkQA
Copy link

SparkQA commented Mar 25, 2016

Test build #54110 has finished for PR 11946 at commit 6479eb4.

  • This patch fails from timeout after a configured wait of 250m.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor Author

rxin commented Mar 25, 2016

Merging in master.

@asfgit asfgit closed this in 3619fec Mar 25, 2016
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