Skip to content

[SPARK-3999][deploy] resolve the wrong number of arguments for pattern error#2842

Closed
baishuo wants to merge 1 commit intoapache:masterfrom
baishuo:testAkka
Closed

[SPARK-3999][deploy] resolve the wrong number of arguments for pattern error#2842
baishuo wants to merge 1 commit intoapache:masterfrom
baishuo:testAkka

Conversation

@baishuo
Copy link
Copy Markdown
Contributor

@baishuo baishuo commented Oct 18, 2014

AssociationErrorEvent which is provided by latest jar(akka-remote_2.10-2.2.3-shaded-protobuf.jar) only have 4 arguments

@AmplabJenkins
Copy link
Copy Markdown

Can one of the admins verify this patch?

@JoshRosen
Copy link
Copy Markdown
Contributor

Jenkins, this is ok to test.

@SparkQA
Copy link
Copy Markdown

SparkQA commented Oct 18, 2014

QA tests have started for PR 2842 at commit ab32894.

  • This patch merges cleanly.

@SparkQA
Copy link
Copy Markdown

SparkQA commented Oct 18, 2014

QA tests have finished for PR 2842 at commit ab32894.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins
Copy link
Copy Markdown

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

@JoshRosen
Copy link
Copy Markdown
Contributor

It looks like this could be a case where the AssociationErrorEvent case-class is backwards-compatible from a Java POV but not w.r.t. Scala pattern-matching. If we have to support multiple Akka binaries here, maybe we can just change this code to access the fields through the object rather than extracting them via pattern matching.

@pwendell
Copy link
Copy Markdown
Contributor

@baishuo are you trying to modify the akka version inside of Spark? In Spark currently support a specific akka version, that version was upgraded recently, but Spark should build just fine (this, in fact, breaks the build).

@baishuo
Copy link
Copy Markdown
Contributor Author

baishuo commented Oct 20, 2014

@JoshRosen @pwendell I know the reason of this problem. In idea, I should right click the project and click maven->reimport

@baishuo baishuo closed this Oct 20, 2014
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.

5 participants