[SPARK-2805] akka 2.3.4

[avati]

Upgrade to akka 2.3.4

[AmplabJenkins]

Can one of the admins verify this patch?

[srowen]

This is the bit that can't be committed now, right? is there a subset of the update that works with Scala 2.10 and 2.11 at the same time? that might be a good bit to break off first.

[avati]

On Thu, Jul 31, 2014 at 1:50 AM, Sean Owen [email protected] wrote:

This is the bit that can't be committed now, right? is there a subset of
the update that works with Scala 2.10 and 2.11 at the same time? that might
be a good bit to break off first.

I am breaking up the patches into as many separate PRs as possible, this is
just one of them.

[avati]

The latest updated patches depend on published packages built by https://github.com/avati/spark-shaded scripts

[ScrapCodes]

This looks good to me, given automated tests pass.

[JoshRosen]

Jenkins, this is ok to test.

[avati]

I guess someone needs to publish the shaded jars first?

[SparkQA]

Can one of the admins verify this patch?

[rbraley]

This is a critical patch for https://github.com/krasserm/akka-analytics/

[pwendell]

@avati would you mind giving this a spin with the staged version of Akka? You'll need to add some remote repositories since we have several akka libraries we are working with:

https://oss.sonatype.org/content/repositories/orgspark-project-1062
...
https://oss.sonatype.org/content/repositories/orgspark-project-1072

[ScrapCodes]

Jenkins, retest this please.

[ScrapCodes]

LGTM, I have tested it locally by running test suits(only relevant ones.) @pwendell Can you trigger jenkins here and should be okay to merge ?

[vanzin]

Also, could you make the PR title a little more descriptive? Just use the description as the title. Since these are used for the final git commit, it's better to have less cryptic information there.

[pwendell]

Jenkins, testt his please.

[pwendell]

Jenkins, test this please.

[pwendell]

Jenkins, test this please.

[SparkQA]

QA tests have started for PR 1685 at commit 57a2315.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 1685 at commit 57a2315.

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

[AmplabJenkins]

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

[pwendell]

This is failing because some zeromq content hasn't been published to maven central. I just published it so we'll give it some time and re-run.

[pwendell]

Jenkins, retest this please.

[SparkQA]

QA tests have started for PR 1685 at commit 57a2315.

• This patch merges cleanly.

[SparkQA]

QA tests have finished for PR 1685 at commit 57a2315.

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

[AmplabJenkins]

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

[pwendell]

Wow - I'm shocked this passed tests right away.

[pwendell]

@avati is there a reason we disabled this test? I think we still want to support the Akka input stream.

[pwendell]

Okay @ScrapCodes is going to add a patch to fix this - I guess we were using some deprecated API's here.

[ScrapCodes]

I have created an issue to track this. https://issues.apache.org/jira/browse/SPARK-3872

[pwendell]

Great LGTM - thanks @avati for the help with this.