[MAPREDUCE-3336] com.google.inject.internal.Preconditions not public api - shouldn't be using it

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 0.23.0
• Fix Version/s: 0.23.1
• Component/s: mrv2
• Target Version/s: 0.23.1
• Hadoop Flags: Reviewed
• Assignee: Thomas Graves
• Reporter: Thomas Graves
• Created: 02/Nov/11 21:35
• Updated: 05/Mar/12 02:48
• Resolved: 09/Nov/11 12:43

Description

com.google.inject.internal.Preconditions does not exist in guice 3.0 and from in guice 2.0 it was an internal api and shouldn't have been used. We should use com.google.common.base.Preconditions instead.

This is currently being used in hadoop-mapreduce-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-nodemanager/src/main/java/org/apache/hadoop/yarn/server/nodemanager/containermanager/monitor/ContainersMonitorImpl.java.

Issue Links

• blocks: MAPREDUCE-2863 Support web-services for RM & NM