Skip to content

Conversation

@yanboliang
Copy link
Contributor

What changes were proposed in this pull request?

#16092 moves YARN resource manager related code to resource-managers/yarn directory. The test case YarnSchedulerBackendSuite was added after that but with the wrong place. I move it to correct directory in this PR.

How was this patch tested?

Existing test.

@SparkQA
Copy link

SparkQA commented Jan 16, 2017

Test build #71427 has started for PR 16595 at commit 9301974.

@SparkQA
Copy link

SparkQA commented Jan 16, 2017

Test build #71433 has finished for PR 16595 at commit 9301974.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • class YarnSchedulerBackendSuite extends SparkFunSuite with MockitoSugar with LocalSparkContext

@yanboliang
Copy link
Contributor Author

cc @rxin @vanzin @srowen

@rxin
Copy link
Contributor

rxin commented Jan 16, 2017

LGTM

@yanboliang
Copy link
Contributor Author

Merged into master. Thanks for reviewing.

@asfgit asfgit closed this in 84f0b64 Jan 17, 2017
@yanboliang yanboliang deleted the yarn branch January 17, 2017 08:30
uzadude pushed a commit to uzadude/spark that referenced this pull request Jan 27, 2017
…n directory.

## What changes were proposed in this pull request?
apache#16092 moves YARN resource manager related code to resource-managers/yarn directory. The test case ```YarnSchedulerBackendSuite``` was added after that but with the wrong place. I move it to correct directory in this PR.

## How was this patch tested?
Existing test.

Author: Yanbo Liang <[email protected]>

Closes apache#16595 from yanboliang/yarn.
cmonkey pushed a commit to cmonkey/spark that referenced this pull request Feb 15, 2017
…n directory.

## What changes were proposed in this pull request?
apache#16092 moves YARN resource manager related code to resource-managers/yarn directory. The test case ```YarnSchedulerBackendSuite``` was added after that but with the wrong place. I move it to correct directory in this PR.

## How was this patch tested?
Existing test.

Author: Yanbo Liang <[email protected]>

Closes apache#16595 from yanboliang/yarn.
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