Skip to content

[Minor][YARN] Move YarnSchedulerBackendSuite to resource-managers/yarn directory. #16595

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

Closed
wants to merge 1 commit into from

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 <ybliang8@gmail.com>

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 <ybliang8@gmail.com>

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