[#1264][FEATURE][SPARK] Support cancel async thread of handle blockEvent and rpc when writer is killed
What changes were proposed in this pull request?
Cancel all the runnable that are wait to be executed or blocked in waiting for rpc callback
Why are the changes needed?
This is helpful when current task is killed since speculation task attempts succeed, but the rpc of which send data still keep retrying
Fix: #1264
Does this PR introduce any user-facing change?
No.
How was this patch tested?
UT
Codecov Report
Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
Project coverage is 51.21%. Comparing base (
8ad0f8d) to head (7b0f563). Report is 1 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #2381 +/- ##
============================================
- Coverage 51.34% 51.21% -0.14%
+ Complexity 3615 3016 -599
============================================
Files 571 481 -90
Lines 32892 23193 -9699
Branches 2833 2140 -693
============================================
- Hits 16890 11878 -5012
+ Misses 14932 10569 -4363
+ Partials 1070 746 -324
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
π New features to boost your workflow:
- β Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- π¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
Test Results
β3β008 filesβ β-β11βββ3β008 suitesβ β-β11βββ6h 38m 27s β±οΈ - 6m 23s β1β178 tests Β±β0βββ1β175 β β-ββ2βββ1 π€ Β±0ββ0 β Β±0ββ2 π₯ +2β 14β911 runsβ β-β11ββ14β893 β β-β14ββ15 π€ Β±0ββ0 β Β±0ββ3 π₯ +3β
For more details on these errors, see this check.
Results for commit be8ca4e7.βΒ± Comparison against base commit 94e211c4.
:recycle: This comment has been updated with latest results.
@jerqi @LuciferYang PTAL
also cc @advancedxy
Seems we should add a new test to cover this
gentle ping @LuciferYang @advancedxy