[HADOOP-10678] SecurityUtil has unnecessary synchronization on collection used for only tests

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 2.4.0, 3.0.0-alpha1
• Fix Version/s: 2.5.0
• Component/s: security
• Target Version/s: 2.5.0
• Hadoop Flags: Reviewed
• Assignee: Benoy Antony
• Reporter: Benoy Antony
• Created: 10/Jun/14 18:48
• Updated: 12/May/16 18:22
• Resolved: 13/Jun/14 16:53

Description

The function SecurityUtil.getKerberosInfo() is a function used during authentication and authorization.

It has two synchronized blocks and one of them is on testProviders. This is an unnecessary lock given that the testProviders is empty in real scenario.

Issue Links

• relates to: MAPREDUCE-2746 [MR-279] [Security] Yarn servers can't communicate with each other with hadoop.security.authorization set to true