Skip to content

Conversation

@sun-rui
Copy link
Contributor

@sun-rui sun-rui commented May 3, 2016

What changes were proposed in this pull request?

Fix warnings and a failure in SparkR test cases with testthat version 1.0.1

How was this patch tested?

SparkR unit test cases.

@SparkQA
Copy link

SparkQA commented May 3, 2016

Test build #57638 has finished for PR 12867 at commit 5c7eec7.

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

@shivaram
Copy link
Contributor

shivaram commented May 3, 2016

LGTM

@shivaram
Copy link
Contributor

shivaram commented May 3, 2016

Merging this to master and branch-2.0

asfgit pushed a commit that referenced this pull request May 3, 2016
… with testthat version 1.0.1

## What changes were proposed in this pull request?
Fix warnings and a failure in SparkR test cases with testthat version 1.0.1

## How was this patch tested?
SparkR unit test cases.

Author: Sun Rui <sunrui2016@gmail.com>

Closes #12867 from sun-rui/SPARK-15091.

(cherry picked from commit 8b6491f)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
@asfgit asfgit closed this in 8b6491f May 3, 2016
asfgit pushed a commit that referenced this pull request Sep 5, 2016
… with testthat version 1.0.1

Fix warnings and a failure in SparkR test cases with testthat version 1.0.1

SparkR unit test cases.

Author: Sun Rui <sunrui2016@gmail.com>

Closes #12867 from sun-rui/SPARK-15091.

(cherry picked from commit 8b6491f)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
zzcclp pushed a commit to zzcclp/spark that referenced this pull request Sep 6, 2016
… with testthat version 1.0.1

Fix warnings and a failure in SparkR test cases with testthat version 1.0.1

SparkR unit test cases.

Author: Sun Rui <sunrui2016@gmail.com>

Closes apache#12867 from sun-rui/SPARK-15091.

(cherry picked from commit 8b6491f)
Signed-off-by: Shivaram Venkataraman <shivaram@cs.berkeley.edu>
(cherry picked from commit 21be94b)
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