Skip to content

SPARK-4770. [DOC] [YARN] spark.scheduler.minRegisteredResourcesRatio doc...#3624

Closed
sryza wants to merge 1 commit intoapache:masterfrom
sryza:sandy-spark-4770
Closed

SPARK-4770. [DOC] [YARN] spark.scheduler.minRegisteredResourcesRatio doc...#3624
sryza wants to merge 1 commit intoapache:masterfrom
sryza:sandy-spark-4770

Conversation

@sryza
Copy link
Contributor

@sryza sryza commented Dec 5, 2014

...umented default is incorrect for YARN

@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24192 has started for PR 3624 at commit bd81a3a.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Dec 5, 2014

Test build #24192 has finished for PR 3624 at commit bd81a3a.

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

@tgravescs
Copy link
Contributor

+1. Thanks Sandy!

@tgravescs
Copy link
Contributor

@pwendell is it ok to pull this doc change into 1.2?

@JoshRosen
Copy link
Contributor

@tgravescs It should be fine to pull docs-only changes into branch-1.2. We're trying to hold off on merging code changes that aren't addressing 1.2.0 release blockers because we don't want to risk introducing new regressions and having to call new votes. If you do want to merge a code change that should eventually be backported into branch-1.2, just merge it into the other branches, leave its JIRA open with 1.2.1 listed in Target Version/s and not Fix Version/s, then add the backport-needed label to the issue so that we remember to come back to it after 1.2.0 is released.

@JoshRosen
Copy link
Contributor

I grepped through the code and this LGTM, too, so I'm going to merge this into master and branch-1.2. Thanks @sryza!

asfgit pushed a commit that referenced this pull request Dec 9, 2014
…doc...

...umented default is incorrect for YARN

Author: Sandy Ryza <sandy@cloudera.com>

Closes #3624 from sryza/sandy-spark-4770 and squashes the following commits:

bd81a3a [Sandy Ryza] SPARK-4770. [DOC] [YARN] spark.scheduler.minRegisteredResourcesRatio documented default is incorrect for YARN

(cherry picked from commit cda94d1)
Signed-off-by: Josh Rosen <joshrosen@databricks.com>
@asfgit asfgit closed this in cda94d1 Dec 9, 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