Skip to content

Conversation

@hhbyyh
Copy link
Contributor

@hhbyyh hhbyyh commented Feb 10, 2015

Trivial. add sc stop and reorganize import
https://issues.apache.org/jira/browse/SPARK-5717

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27198 has started for PR 4503 at commit 2e85cc1.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27198 has finished for PR 4503 at commit 2e85cc1.

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

@AmplabJenkins
Copy link

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27198/
Test FAILed.

@hhbyyh
Copy link
Contributor Author

hhbyyh commented Feb 10, 2015

The ut failure seems irrelevant. Not sure how to trigger the retest properly.

Copy link
Member

Choose a reason for hiding this comment

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

The Spark imports were correctly ordered already. I think the scopt import could go after the log4j import, if we're matching style of other imports.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK.

@srowen
Copy link
Member

srowen commented Feb 10, 2015

Yes it's a small thing but the examples should call stop(). I can merge if you'll move the imports accordingly.

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27208 has started for PR 4503 at commit 7837a2c.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Feb 10, 2015

jenkins, retest this please

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27209 has started for PR 4503 at commit 7837a2c.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27208 has finished for PR 4503 at commit 7837a2c.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27208/
Test PASSed.

@SparkQA
Copy link

SparkQA commented Feb 10, 2015

Test build #27209 has finished for PR 4503 at commit 7837a2c.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27209/
Test PASSed.

asfgit pushed a commit that referenced this pull request Feb 10, 2015
Trivial. add sc stop and reorganize import
https://issues.apache.org/jira/browse/SPARK-5717

Author: JqueryFan <firing@126.com>
Author: Yuhao Yang <hhbyyh@gmail.com>

Closes #4503 from hhbyyh/scstop and squashes the following commits:

7837a2c [JqueryFan] revert import change
2e85cc1 [Yuhao Yang] add stop and reorganize import

(cherry picked from commit 6cc96cf)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@asfgit asfgit closed this in 6cc96cf Feb 10, 2015
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