[CURATOR-208] InterProcessSemaphoreV2 swallows InterruptedException

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.6.0, 2.7.1
• Fix Version/s: 3.1.0, 2.10.0
• Component/s: Recipes
• Labels: easyfix
• Assignee: Jordan Zimmerman
• Reporter: Alexei Osipov
• Created: 10/Apr/15 09:53
• Updated: 18/Jan/16 00:47
• Resolved: 18/Jan/16 00:47

Description

InterProcessSemaphoreV2 incorrectly processes InterruptedException.

Method `InterProcessSemaphoreV2#makeLease(final String path)` contains code block

                try
                {
                    client.delete().guaranteed().forPath(path);
                }
                catch ( KeeperException.NoNodeException e )
                {
                    log.warn("Lease already released", e);
                }
                catch ( Exception e )
                {
                    throw new IOException(e);
                }

The problem is that code in try block may throw an InterruptedException and this exception gets wrapped into IOException so it becomes very problematic to handle it properly.

Issue Links

• supercedes: CURATOR-109 Threads that are sleeping should get interrupted when closing the client.