[HADOOP-7900] LocalDirAllocator confChanged() accesses conf.get() twice

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.23.1, 0.24.0
• Fix Version/s: 2.0.0-alpha
• Component/s: fs
• Target Version/s: 0.23.3, 0.24.0
• Hadoop Flags: Reviewed
• Assignee: Ravi Gummadi
• Reporter: Ravi Gummadi
• Created: 09/Dec/11 17:03
• Updated: 23/May/12 20:15
• Resolved: 11/Mar/12 20:45

Description

LocalDirAllocator.AllocatorPerContext.confChanged() accesses conf.get() twice unnecessarily. The 2 calls can give 2 different values, which can lead to issues because the first call's return value is saved in savedLocalDirs and is used for comparison in the next call to confChanged() method — So the comparison is wrong.