[FLINK-22886] Thread leak in RocksDBStateUploader

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.11.3, 1.13.1, 1.12.4
• Fix Version/s: 1.14.0, 1.12.5, 1.13.2
• Component/s: Runtime / State Backends
• Labels: critical pull-request-available
• Assignee: Yue Ma
• Reporter: Jiayi Liao
• Created: 06/Jun/21 05:45
• Updated: 28/Aug/21 12:19
• Resolved: 22/Jun/21 18:27

Description

ExecutorService in RocksDBStateUploader is not shut down, which may leak thread when tasks fail.

BTW, we should name the thread group in ExecutorService, otherwise what we see in the stack, is a lot of threads named with pool-m-thread-n like this:

 

Issue Links

• relates to: FLINK-23003 Resource leak in RocksIncrementalSnapshotStrategy
• links to: GitHub Pull Request #16171
• links to: GitHub Pull Request #16234