spark icon indicating copy to clipboard operation
spark copied to clipboard

[SPARK-53622][CORE][TEST] Improve `UninterruptibleThreadSuite`

Open vrozov opened this issue 4 months ago • 3 comments

What changes were proposed in this pull request?

Implement several code improvement in UninterruptibleThreadSuite:

  • run several iterations of stress test
  • log InterruptedException
  • improve assert error message ("false did not equal true" => "hasInterruptedException was false")
  • use await instead of sleep
  • fail test fast

Why are the changes needed?

improve test coverage of UninterruptibleThread and help with test failure troubleshooting

Does this PR introduce any user-facing change?

No

How was this patch tested?

Run modified test

Was this patch authored or co-authored using generative AI tooling?

No

vrozov avatar Sep 17 '25 19:09 vrozov

@cloud-fan @Ngone51 Please take a look

vrozov avatar Sep 17 '25 19:09 vrozov

@cloud-fan @Ngone51 Please take a look

vrozov avatar Sep 29 '25 15:09 vrozov

@cloud-fan Please take a look

vrozov avatar Oct 21 '25 04:10 vrozov