Skip to content

Conversation

@felixcheung
Copy link
Member

@rxin @davies @shivaram
Took save mode from my PR #10480, and move everything to writer methods. This is related to PR #10559

  • it seems jsonRDD() is broken, need to investigate - this is not a public API though; will look into some more tonight. (fixed)

@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #48708 has finished for PR 10584 at commit e0e2a9c.

  • This patch fails R style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@rxin
Copy link
Contributor

rxin commented Jan 5, 2016

Thanks for submitting this.

FYI #10559 was merged, and R tests were disabled there so you should re-enable them.

@shivaram
Copy link
Contributor

shivaram commented Jan 5, 2016

Thanks @felixcheung - LGTM if tests pass. Also @davies will be good if you can take a look.

@felixcheung
Copy link
Member Author

I found another one after rebasing on #10559
also fixed jsonRDD

@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #48722 has finished for PR 10584 at commit 203a52f.

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

@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #48724 has finished for PR 10584 at commit d8d9b21.

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

@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #2320 has finished for PR 10584 at commit dc59f84.

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

@rxin
Copy link
Contributor

rxin commented Jan 5, 2016

Thanks - merging this in master.

@asfgit asfgit closed this in cc4d522 Jan 5, 2016
@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #48732 has finished for PR 10584 at commit dc59f84.

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

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.

4 participants