Skip to content

Conversation

@aarondav
Copy link
Contributor

@aarondav aarondav commented Nov 1, 2014

Note that we're turning this on for at least the first part of the QA period as a trial. We want to enable this (and deprecate the NioBlockTransferService) as soon as possible in the hopes that NettyBlockTransferService will be more stable and easier to maintain. We will turn it off if we run into major issues.

@SparkQA
Copy link

SparkQA commented Nov 1, 2014

Test build #22698 has started for PR 3049 at commit bb981cc.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Nov 1, 2014

Test build #22698 has finished for PR 3049 at commit bb981cc.

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

@pwendell
Copy link
Contributor

pwendell commented Nov 1, 2014

LGTM - as discussed offline let's send an email to the dev list to make them aware of this change.

@pwendell
Copy link
Contributor

pwendell commented Nov 1, 2014

Okay I'll pull this in and send an e-mail.

@asfgit asfgit closed this in 59e626c Nov 1, 2014
guavuslabs-builder pushed a commit to ThalesGroup/spark that referenced this pull request Nov 3, 2014
Note that we're turning this on for at least the first part of the QA period as a trial. We want to enable this (and deprecate the NioBlockTransferService) as soon as possible in the hopes that NettyBlockTransferService will be more stable and easier to maintain. We will turn it off if we run into major issues.

Author: Aaron Davidson <[email protected]>

Closes apache#3049 from aarondav/enable-netty and squashes the following commits:

bb981cc [Aaron Davidson] [SPARK-4183] Enable NettyBlockTransferService by default
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