[HBASE-10497] Correct wrong handling and add proper handling for swallowed InterruptedException thrown by Thread.sleep under HBase-Client/HBase-Server folders systematically

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Abandoned
• Component/s: Client, regionserver
• Assignee: Unassigned
• Reporter: Honghua Feng
• Created: 11/Feb/14 09:51
• Updated: 17/Jun/22 15:41
• Resolved: 17/Jun/22 15:41

Description

There are two kinds of handling problems for InterruptedException thrown by Thread.sleep in many places under HBase-Client/HBase-Server folders:

1. Thread.currentThread.interrupt() is called within 'while' loops which can result in buggy behaviors such as expected sleep doesn't occur due to restored interrupt status by former loop
2. InterruptedException thrown by Thread.sleep are swallowed silently (which are neither declared in the caller method's throws clause nor rethrown immediately)

Sub-Tasks

• Add handling for swallowed InterruptedException thrown by Thread.sleep in rest related files: Closed
• Add handling for swallowed InterruptedException thrown by Thread.sleep in MiniZooKeeperCluster: Closed
• Add handling for swallowed InterruptedException thrown by Thread.sleep in RpcServer: Closed
• Correct wrong handling and add proper handling for swallowed InterruptedException thrown by Thread.sleep in client: Closed
• Correct wrong handling and add proper handling for swallowed InterruptedException thrown by Thread.sleep in util: Closed
• Correct wrong handling and add proper handling for swallowed InterruptedException thrown by Thread.sleep in regionserver: Closed