Skip to content

Conversation

@ryan-williams
Copy link
Contributor

This helped me to debug a parse error that was due to the event log format changing recently.

@SparkQA
Copy link

SparkQA commented Mar 21, 2015

Test build #28949 has started for PR 5122 at commit c3742ae.

  • This patch merges cleanly.

@srowen
Copy link
Member

srowen commented Mar 21, 2015

LGTM. The exception is otherwise completely swallowed, and this is supposed to be treated as an error.

@SparkQA
Copy link

SparkQA commented Mar 21, 2015

Test build #28949 has finished for PR 5122 at commit c3742ae.

  • This patch fails Scala style 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/28949/
Test FAILed.

@ryan-williams
Copy link
Contributor Author

oof, line length, fixing

@SparkQA
Copy link

SparkQA commented Mar 21, 2015

Test build #28951 has started for PR 5122 at commit 5831656.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Mar 21, 2015

Test build #28951 has finished for PR 5122 at commit 5831656.

  • 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/28951/
Test PASSed.

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