Skip to content

[SPARK-32996][Web-UI][FollowUP] Move ExecutorSummarySuite to proper path#29926

Closed
shrutig wants to merge 4 commits intoapache:masterfrom
shrutig:SPARK-32996
Closed

[SPARK-32996][Web-UI][FollowUP] Move ExecutorSummarySuite to proper path#29926
shrutig wants to merge 4 commits intoapache:masterfrom
shrutig:SPARK-32996

Conversation

@shrutig
Copy link
Contributor

@shrutig shrutig commented Oct 1, 2020

What changes were proposed in this pull request?

This change updates the test file location in #29872 to proper path.

Why are the changes needed?

ExecutorSummarySuite.scala should be in core/src/test/scala instead of core/src/test/java.

Does this PR introduce any user-facing change?

No

How was this patch tested?

Unit tests

@viirya viirya changed the title SPARK-32996][Web-UI][3.0] Handle empty ExecutorMetrics in ExecutorMetricsJsonSerializer [SPARK-32996][Web-UI][FollowUP] Handle empty ExecutorMetrics in ExecutorMetricsJsonSerializer Oct 1, 2020
@viirya
Copy link
Member

viirya commented Oct 1, 2020

ok to test

@viirya viirya changed the title [SPARK-32996][Web-UI][FollowUP] Handle empty ExecutorMetrics in ExecutorMetricsJsonSerializer [SPARK-32996][Web-UI][FollowUP] Move ExecutorSummarySuite to proper path Oct 1, 2020
@SparkQA
Copy link

SparkQA commented Oct 1, 2020

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/33932/

@viirya
Copy link
Member

viirya commented Oct 1, 2020

look good pending test.

@SparkQA
Copy link

SparkQA commented Oct 1, 2020

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/33932/

@SparkQA
Copy link

SparkQA commented Oct 1, 2020

Test build #129319 has finished for PR 29926 at commit 5edeb0f.

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

@viirya
Copy link
Member

viirya commented Oct 1, 2020

Thanks! Merging to master.

@SparkQA
Copy link

SparkQA commented Oct 1, 2020

Test build #129318 has finished for PR 29926 at commit 5edeb0f.

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

@viirya viirya closed this in 8657742 Oct 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants