Skip to content
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.

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

[SPARK-11686][CORE] Issue WARN when dynamic allocation is disabled due to spark.dynamicAllocation.enabled and spark.executor.instances both set #9926

Closed
wants to merge 1 commit into from

Conversation

toozol
Copy link

@toozol toozol commented Nov 24, 2015

Changed the log type to a 'warning' instead of 'info' as required.

…ark.dynamicAllocation.enabled and spark.executor.instances both set
@srowen
Copy link
Member

srowen commented Nov 24, 2015

@ashwinswaroop please read https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark The title here is insufficient, for example. I think the change is trivial but fine.

@SparkQA
Copy link

SparkQA commented Nov 24, 2015

Test build #2101 has started for PR 9926 at commit ea94295.

@toozol toozol changed the title SPARK-11686 [SPARK-11686][CORE] Issue WARN when dynamic allocation is disabled due to spark.dynamicAllocation.enabled and spark.executor.instances both set Nov 25, 2015
@toozol
Copy link
Author

toozol commented Nov 25, 2015

@srowen Sorry for the incorrect title format. I have changed it accordingly.

@SparkQA
Copy link

SparkQA commented Nov 25, 2015

Test build #2110 has finished for PR 9926 at commit ea94295.

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

asfgit pushed a commit that referenced this pull request Nov 25, 2015
…e to spark.dynamicAllocation.enabled and spark.executor.instances both set

Changed the log type to a 'warning' instead of 'info' as required.

Author: Ashwin Swaroop <Ashwin Swaroop>

Closes #9926 from ashwinswaroop/master.

(cherry picked from commit 6385002)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen
Copy link
Member

srowen commented Nov 25, 2015

Merged to master/1.6

@asfgit asfgit closed this in 6385002 Nov 25, 2015
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.

3 participants