Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Jan 4, 2018

No description provided.

@vanzin
Copy link
Contributor Author

vanzin commented Jan 4, 2018

@foxish @liyinan926

name="kubernetes",
dependencies=[],
source_file_regexes=["resource-managers/kubernetes/core"],
source_file_regexes=["resource-managers/kubernetes"],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise when you change something outside of kubernetes/core (like the docker files), all unit tests will be run, not just the stuff related to k8s.

I requested this change in the original PR but you guys never addressed it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, sorry the comment was not caught by us.

@liyinan926
Copy link
Contributor

LGTM.

@SparkQA
Copy link

SparkQA commented Jan 4, 2018

Test build #85691 has finished for PR 20156 at commit a09a728.

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

@vanzin
Copy link
Contributor Author

vanzin commented Jan 4, 2018

Merging to master / 2.3.

asfgit pushed a commit that referenced this pull request Jan 4, 2018
Author: Marcelo Vanzin <[email protected]>

Closes #20156 from vanzin/SPARK-22948.

(cherry picked from commit 95f9659)
Signed-off-by: Marcelo Vanzin <[email protected]>
@asfgit asfgit closed this in 95f9659 Jan 4, 2018
@SparkQA
Copy link

SparkQA commented Jan 4, 2018

Test build #85692 has finished for PR 20156 at commit edc3e18.

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

@vanzin vanzin deleted the SPARK-22948 branch January 5, 2018 22:35
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