Skip to content

Conversation

@dgshep
Copy link
Contributor

@dgshep dgshep commented Apr 27, 2017

What changes were proposed in this pull request?

Add test case for scenarios where executor.cores is set as a
(non)divisor of spark.cores.max
This tests the change in
#17786

How was this patch tested?

Ran the existing test suite with the new tests

@dbtsai

Add test case for scenarios where executor.cores is set as a
(non)divisor of spark.cores.max
@dgshep
Copy link
Contributor Author

dgshep commented Apr 27, 2017

Jenkins, ok to test.

@dgshep dgshep changed the title [SPARK-20483][MINOR] Add test for case [SPARK-20483][MINOR] Add test cases where executor.cores is set Apr 27, 2017
@dgshep dgshep changed the title [SPARK-20483][MINOR] Add test cases where executor.cores is set [SPARK-20483][MINOR] Test for Mesos Coarse mode may starve other Mesos frameworks Apr 27, 2017
@SparkQA
Copy link

SparkQA commented Apr 27, 2017

Test build #76240 has finished for PR 17788 at commit 7270fef.

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

@dbtsai
Copy link
Member

dbtsai commented Apr 27, 2017

LGTM. Merged into master and branch 2.1. Thanks.

asfgit pushed a commit that referenced this pull request Apr 27, 2017
…s frameworks

## What changes were proposed in this pull request?

Add test case for scenarios where executor.cores is set as a
(non)divisor of spark.cores.max
This tests the change in
#17786

## How was this patch tested?

Ran the existing test suite with the new tests

dbtsai

Author: Davis Shepherd <[email protected]>

Closes #17788 from dgshep/add_mesos_test.

(cherry picked from commit 039e32c)
Signed-off-by: DB Tsai <[email protected]>
@asfgit asfgit closed this in 039e32c Apr 27, 2017
@dgshep dgshep deleted the add_mesos_test branch April 27, 2017 20:58
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