Skip to content

Conversation

WangTaoTheTonic
Copy link
Contributor

https://issues.apache.org/jira/browse/SPARK-3565

"spark.ports.maxRetries" should be "spark.port.maxRetries". Make the configuration keys in document and code consistent.

@SparkQA
Copy link

SparkQA commented Sep 17, 2014

QA tests have started for PR 2427 at commit 3700dba.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 17, 2014

QA tests have finished for PR 2427 at commit 3700dba.

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

@WangTaoTheTonic
Copy link
Contributor Author

I am not clear why there is an test failed in org.apache.spark.broadcast.BroadcastSuite, but it seems like it has nothing to do with this commit.
Could we retest it?

@andrewor14
Copy link
Contributor

LGTM, retest this please!

@WangTaoTheTonic
Copy link
Contributor Author

@andrewor14 Looks like Jenkins is not triggered?

@andrewor14
Copy link
Contributor

test this please

@WangTaoTheTonic
Copy link
Contributor Author

He might be tired. -_-

@SparkQA
Copy link

SparkQA commented Sep 18, 2014

QA tests have started for PR 2427 at commit c178813.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Sep 18, 2014

QA tests have finished for PR 2427 at commit c178813.

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

@pwendell
Copy link
Contributor

Yeah good catch, thanks. I'll merge this.

guavuslabs-builder pushed a commit to ThalesGroup/spark that referenced this pull request Sep 18, 2014
https://issues.apache.org/jira/browse/SPARK-3565

"spark.ports.maxRetries" should be "spark.port.maxRetries". Make the configuration keys in document and code consistent.

Author: WangTaoTheTonic <[email protected]>

Closes apache#2427 from WangTaoTheTonic/fixPortRetries and squashes the following commits:

c178813 [WangTaoTheTonic] Use blank lines trigger Jenkins
646f3fe [WangTaoTheTonic] also in SparkBuild.scala
3700dba [WangTaoTheTonic] Fix configuration item not consistent with document

(cherry picked from commit 3f169bf)
Signed-off-by: Patrick Wendell <[email protected]>
@asfgit asfgit closed this in 3f169bf Sep 18, 2014
@WangTaoTheTonic WangTaoTheTonic deleted the fixPortRetries branch September 18, 2014 05:50
markhamstra pushed a commit to markhamstra/spark that referenced this pull request Sep 19, 2014
https://issues.apache.org/jira/browse/SPARK-3565

"spark.ports.maxRetries" should be "spark.port.maxRetries". Make the configuration keys in document and code consistent.

Author: WangTaoTheTonic <[email protected]>

Closes apache#2427 from WangTaoTheTonic/fixPortRetries and squashes the following commits:

c178813 [WangTaoTheTonic] Use blank lines trigger Jenkins
646f3fe [WangTaoTheTonic] also in SparkBuild.scala
3700dba [WangTaoTheTonic] Fix configuration item not consistent with document

(cherry picked from commit 3f169bf)
Signed-off-by: Patrick Wendell <[email protected]>
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