Skip to content

[SPARKR] add csv tests #13904

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
felixcheung wants to merge 2 commits into from
Closed

[SPARKR] add csv tests #13904

felixcheung wants to merge 2 commits into from

Conversation

@felixcheung
Copy link
Member

@felixcheung felixcheung commented Jun 25, 2016

What changes were proposed in this pull request?

Add unit tests for csv data for SPARKR

How was this patch tested?

unit tests

@SparkQA
Copy link

SparkQA commented Jun 25, 2016

Test build #61226 has finished for PR 13904 at commit e3c7a9a.

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

@SparkQA
Copy link

SparkQA commented Jun 26, 2016

Test build #61246 has finished for PR 13904 at commit 296c2d4.

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

@felixcheung
Copy link
Member Author

felixcheung commented Jun 28, 2016

@shivaram I think this will be good to have, just a small change. Thanks!

@shivaram
Copy link
Contributor

shivaram commented Jun 29, 2016

Sure - LGTM. Merging this to master, branch-2.0

asfgit pushed a commit that referenced this pull request Jun 29, 2016
@felixcheung @shivaram
[SPARKR] add csv tests
d7a59f1 
## What changes were proposed in this pull request?

Add unit tests for csv data for SPARKR

## How was this patch tested?

unit tests

Author: Felix Cheung <felixcheung_m@hotmail.com>

Closes #13904 from felixcheung/rcsv.

(cherry picked from commit 823518c)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgit asfgit closed this in 823518c Jun 29, 2016
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

@felixcheung @SparkQA @shivaram