[ACCUMULO-855] Patch to remove acquisition of redundant lock: LiveTServerSet in org.apache.accumulo.server.master

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.5.0
• Assignee: Tim Halloran
• Reporter: Tim Halloran
• Created: 14/Nov/12 18:23
• Updated: 15/Nov/12 17:27
• Resolved: 14/Nov/12 18:35

Description

A lock on the receiver is acquired twice by this method. This has two drawbacks (a) it slows performance and (b) is confusing.

A patch is attached proposing the fix – I left the method synchronized and removed the lock acquisition in the method to match the other methods in the class.