Skip to content

Port Netflix/Hystrix#1976 remove canceled timer tasks#2

Open
amrith92 wants to merge 1 commit into
masterfrom
port-netflix-1976-remove-on-cancel-policy
Open

Port Netflix/Hystrix#1976 remove canceled timer tasks#2
amrith92 wants to merge 1 commit into
masterfrom
port-netflix-1976-remove-on-cancel-policy

Fixed possible memory leak in ScheduledThreadPoolExecutor

8d9bfe8
Select commit
Loading
Failed to load commit list.
Tully Review / Tully review succeeded May 27, 2026 in 0s

Tully review

Looks good as a faithful upstream port.

Only compatibility check: setRemoveOnCancelPolicy(true) requires Java 7+. If this fork's minimum runtime is Java 7+, merge as-is; if Java 6 is still supported, gate it reflectively.

Focused verification is sufficient for this scoped port. A regression test or explicit upstream reference would be nice, not blocking.