Skip to content

SPARK-5930 [DOCS] Documented default of spark.shuffle.io.retryWait is confusing #4769

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
srowen wants to merge 1 commit into from
Closed

SPARK-5930 [DOCS] Documented default of spark.shuffle.io.retryWait is confusing #4769

srowen wants to merge 1 commit into from

Conversation

@srowen
Copy link
Member

@srowen srowen commented Feb 25, 2015

Clarify default max wait in spark.shuffle.io.retryWait docs

CC @andrewor14

@SparkQA
Copy link

SparkQA commented Feb 25, 2015

Test build #27952 has started for PR 4769 at commit ae2792b.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 25, 2015

Test build #27952 has finished for PR 4769 at commit ae2792b.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 25, 2015

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

@srowen
Copy link
Member Author

srowen commented Feb 25, 2015

retest this please

@SparkQA
Copy link

SparkQA commented Feb 25, 2015

Test build #27954 has started for PR 4769 at commit ae2792b.

  • This patch merges cleanly.

@rxin
Copy link
Contributor

rxin commented Feb 25, 2015

Jenkins doesn't run anything for docs so we don't need to trigger/wait for Jenkins here.

@rxin
Copy link
Contributor

rxin commented Feb 25, 2015

I've merged this.

asfgit pushed a commit that referenced this pull request Feb 25, 2015
@srowen @rxin
SPARK-5930 [DOCS] Documented default of spark.shuffle.io.retryWait is…
eaffc6e 
… confusing

Clarify default max wait in spark.shuffle.io.retryWait docs

CC andrewor14

Author: Sean Owen <[email protected]>

Closes #4769 from srowen/SPARK-5930 and squashes the following commits:

ae2792b [Sean Owen] Clarify default max wait in spark.shuffle.io.retryWait docs

(cherry picked from commit 7d8e6a2)
Signed-off-by: Reynold Xin <[email protected]>
@asfgit asfgit closed this in 7d8e6a2 Feb 25, 2015
@srowen
Copy link
Member Author

srowen commented Feb 25, 2015

Yeah, actually just dumb habit. For non-trivial doc changes I try the docs build locally to sniff out syntax errors

@SparkQA
Copy link

SparkQA commented Feb 25, 2015

Test build #27954 has finished for PR 4769 at commit ae2792b.

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

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 25, 2015

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

@srowen srowen deleted the SPARK-5930 branch February 26, 2015 10:51
@andrewor14 andrewor14 mentioned this pull request Apr 13, 2015
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@srowen @SparkQA @AmplabJenkins @rxin