[MAPREDUCE-5613] DefaultSpeculator holds and checks hashmap that is always empty

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 2.3.0
• Component/s: applicationmaster
• Hadoop Flags: Reviewed
• Assignee: Gera Shegalov
• Reporter: Gera Shegalov
• Created: 07/Nov/13 23:38
• Updated: 24/Feb/14 20:57
• Resolved: 13/Nov/13 09:27

Description

The only way pendingSpeculations is used:

     // If the task is already known to be speculation-bait, don't do anything   
      if (pendingSpeculations.get(task) != null) {                                
        if (pendingSpeculations.get(task).get()) {                                
          return;                                                                 
        }                                                                         
      }