[FLINK-32029] FutureUtils.handleUncaughtException swallows exceptions that are caused by the exception handler code

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.17.0, 1.16.1, 1.18.0
• Fix Version/s: 1.16.2, 1.18.0, 1.17.1
• Component/s: Runtime / Coordination
• Labels: pull-request-available
• Assignee: Matthias Pohl
• Reporter: Matthias Pohl
• Created: 08/May/23 09:18
• Updated: 10/May/23 16:15
• Resolved: 10/May/23 16:15

Description

I ran into an issue in FLINK-31773 where the passed exception handler relies on the leaderContender field of the DefaultLeaderElectionService. This field can become null with the changes of FLINK-31773. But the null check was missed in the error handling code. This bug wasn't exposed because FutureUtils.handleUncaughtException(..) expects the passed error handler callback to be bug-free.

Issue Links

• links to: GitHub Pull Request #22544
• links to: GitHub Pull Request #22561
• links to: GitHub Pull Request #22562