Skip to content

Conversation

@kayousterhout
Copy link
Contributor

The named parameter in an overridden class isn't supported in Scala 2.10 so was breaking the build.

cc @zsxwing

@zsxwing
Copy link
Member

zsxwing commented Oct 25, 2016

LGTM. The compile is successful. I'm going to merge it. Thanks!

@asfgit asfgit closed this in 483c37c Oct 25, 2016
@SparkQA
Copy link

SparkQA commented Oct 25, 2016

Test build #67481 has finished for PR 15617 at commit c09ec20.

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

@kayousterhout
Copy link
Contributor Author

Thanks @zsxwing -- sorry about this! Looks like the test failure was unrelated.

@zsxwing
Copy link
Member

zsxwing commented Oct 25, 2016

@kayousterhout no problem. yeah, the failure is unrelated.

@erenavsarogullari
Copy link
Member

Thanks @kayousterhout and @zsxwing for the support. I have missed to build with v2.10 and will help as a good experience for incoming PRs ;)

robert3005 pushed a commit to palantir/spark that referenced this pull request Nov 1, 2016
The named parameter in an overridden class isn't supported in Scala 2.10 so was breaking the build.

cc zsxwing

Author: Kay Ousterhout <[email protected]>

Closes apache#15617 from kayousterhout/hotfix.
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
The named parameter in an overridden class isn't supported in Scala 2.10 so was breaking the build.

cc zsxwing

Author: Kay Ousterhout <[email protected]>

Closes apache#15617 from kayousterhout/hotfix.
@kayousterhout kayousterhout deleted the hotfix branch April 12, 2017 00:46
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