[HIVE-18966] LLAP should not shut down when some random thread goes down

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned Assign to me
• Reporter: Sergey Shelukhin
• Created: 15/Mar/18 17:58
• Updated: 15/Mar/18 18:01

Description

LlapDaemonUncaughtExceptionHandler shuts down LLAP on any thread failure.
Not sure it's the best idea... 3rd party libraries like permitted UDFs or Storage Handlers (e.g. Druid recently) can have errors that should not bring the entire daemon down.
Perhaps we can go by thread name pattern?
Overall logging the error, unless it's an OOM or other Error, might be better.

We can also add error handling to important threads like schedulers, if it's missing, that will convert an exception into some critical one that will tell the handler to shut everything down.

Issue Links

• is related to: HIVE-18959 Avoid creating extra pool of threads within LLAP