[HBASE-15080] Remove synchronized block from MasterServiceStubMaker#releaseZooKeeperWatcher()

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.98.17
• Hadoop Flags: Reviewed
• Assignee: Ted Yu
• Reporter: Ted Yu
• Created: 08/Jan/16 00:24
• Updated: 09/Jan/16 17:01
• Resolved: 08/Jan/16 18:14

Description

This is a follow up to HBASE-11460

elserj found that in 0.98, the synchronized block below should have been taken out (as was done for branch-1 +):

      synchronized (masterAndZKLock) {
        if (keepAliveZookeeperUserCount.decrementAndGet() <= 0 ){

keepAliveZookeeperUserCount is an AtomicInteger. There is no need for the synchronized block.

This issue is to remove the synchronized block.

Issue Links

• relates to: HBASE-11460 Deadlock in HMaster on masterAndZKLock in HConnectionManager