[THRIFT-4729] Crash: ForceClose expired task with Exception uncaught

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8
• Fix Version/s: 0.10.0
• Component/s: C++ - Library
• Assignee: James E. King III
• Reporter: KG.Xu
• Created: 08/Jan/19 01:15
• Updated: 14/Jan/19 15:31
• Resolved: 14/Jan/19 15:31

Description

Context is: I've employed NonblockingServer & ThreadManager, with setTaskExpireTime() used in my app. Under heavy load, it crashed. Crash point is forceClose()::throw TException (to be specific, send(NotificationSocketFD) failed).
I went through the code, noted that during removeExpiredTasks() there is no try-catch involved. On the contrary, Task::run() is enclosed by try-catch.

Is it by design, or just a bug ? Thanks.