Skip to content

Conversation

@zero323
Copy link
Member

@zero323 zero323 commented Nov 16, 2015

The goal of this PR is to add tests covering the issue to ensure that is was resolved by SPARK-11086.

@shivaram
Copy link
Contributor

Jenkins, ok to test

@SparkQA
Copy link

SparkQA commented Nov 16, 2015

Test build #46017 has finished for PR 9743 at commit 7c2cb03.

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

Copy link
Contributor

Choose a reason for hiding this comment

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

how about
expect_identical(ldf[, 1], collected[, 1])?

@SparkQA
Copy link

SparkQA commented Nov 17, 2015

Test build #46089 has finished for PR 9743 at commit 0508988.

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

Copy link
Contributor

Choose a reason for hiding this comment

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

style: a = 1, b = 2; c = 3

@sun-rui
Copy link
Contributor

sun-rui commented Nov 18, 2015

LGTM except a minor comment on coding style

@SparkQA
Copy link

SparkQA commented Nov 18, 2015

Test build #46201 has finished for PR 9743 at commit 6d2be57.

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

@SparkQA
Copy link

SparkQA commented Nov 18, 2015

Test build #46212 has finished for PR 9743 at commit c7ff1f2.

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

@shivaram
Copy link
Contributor

LGTM. Thanks @zero323 @sun-rui -- Merging this

@asfgit asfgit closed this in a97d6f3 Nov 18, 2015
asfgit pushed a commit that referenced this pull request Nov 18, 2015
The goal of this PR is to add tests covering the issue to ensure that is was resolved by [SPARK-11086](https://issues.apache.org/jira/browse/SPARK-11086).

Author: zero323 <[email protected]>

Closes #9743 from zero323/SPARK-11281-tests.

(cherry picked from commit a97d6f3)
Signed-off-by: Shivaram Venkataraman <[email protected]>
@zero323 zero323 deleted the SPARK-11281-tests branch April 6, 2017 11:03
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