Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

What changes were proposed in this pull request?

This PR aims to add WorkerDecomissionExtendedSuite for various worker decommission combinations.

Why are the changes needed?

This will improve the test coverage.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the Jenkins.

@dongjoon-hyun
Copy link
Member Author

cc @dbtsai and @holdenk

@dongjoon-hyun dongjoon-hyun marked this pull request as draft June 25, 2020 20:02
@SparkQA

This comment has been minimized.

@dongjoon-hyun dongjoon-hyun marked this pull request as ready for review June 25, 2020 21:34
@dongjoon-hyun dongjoon-hyun changed the title [SPARK-32100][CORE] Add WorkerDecommissionExtendedSuite [SPARK-32100][CORE][TESTS] Add WorkerDecommissionExtendedSuite Jun 25, 2020
@SparkQA

This comment has been minimized.

@SparkQA
Copy link

SparkQA commented Jun 25, 2020

Test build #124519 has finished for PR 28929 at commit 6bc677e.

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

@holdenk
Copy link
Contributor

holdenk commented Jun 25, 2020

LGTM pending jenkins, better test coverage for stuff in core is good :) Thanks @dongjoon-hyun :)

@dongjoon-hyun
Copy link
Member Author

Thank you, @holdenk !

@dongjoon-hyun
Copy link
Member Author

All Scala/Java/PySpark test passed, current R tests are running.
Merged to master for Apache Spark 3.1.0.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-WD-TEST branch June 25, 2020 23:22
@SparkQA
Copy link

SparkQA commented Jun 25, 2020

Test build #124522 has finished for PR 28929 at commit 3da70ec.

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

holdenk pushed a commit to holdenk/spark that referenced this pull request Oct 27, 2020
### What changes were proposed in this pull request?

This PR aims to add `WorkerDecomissionExtendedSuite` for various worker decommission combinations.

### Why are the changes needed?

This will improve the test coverage.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the Jenkins.

Closes apache#28929 from dongjoon-hyun/SPARK-WD-TEST.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants