Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Aug 18, 2017

What changes were proposed in this pull request?

Add more cases we should view as a normal query stop rather than a failure.

How was this patch tested?

The new unit tests.

@SparkQA
Copy link

SparkQA commented Aug 18, 2017

Test build #80859 has finished for PR 18997 at commit 8aa3a29.

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

@SparkQA
Copy link

SparkQA commented Aug 20, 2017

Test build #80901 has finished for PR 18997 at commit 8aa3a29.

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

@SparkQA
Copy link

SparkQA commented Aug 24, 2017

Test build #81058 has finished for PR 18997 at commit bbb0b0e.

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

@tdas
Copy link
Contributor

tdas commented Aug 24, 2017

LGTM. Merging to master.

@asfgit asfgit closed this in d3abb36 Aug 24, 2017
@zsxwing zsxwing deleted the SPARK-21788 branch August 24, 2017 18:02
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