[ZEPPELIN-534] Broken RemoteInterpreterService.Client is reused, not discarded

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.5.6
• Assignee: Jungtaek Lim
• Reporter: Jungtaek Lim
• Created: 27/Dec/15 00:28
• Updated: 05/Jan/16 09:22
• Resolved: 05/Jan/16 09:22

Description

Looking into the Zeppelin code, there're getClient() and releaseClient() in RemoteInterpreterProcess, but RemoteInterpreterProcess doesn't expose releaseBrokenClient() which discards broken client from pool.

This means Zeppelin always return Client to pool whether it is broken or not, so broken Client instances are reused or even leaked if releaseClient() is not called.