svn commit: r1077761 - in /hadoop/mapreduce/trunk: CHANGES.txt src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java

[tomwhite]

Author: tomwhite
Date: Fri Mar  4 04:52:35 2011
New Revision: 1077761

URL: http://svn.apache.org/viewvc?rev=1077761&view=rev
Log:
MAPREDUCE-1242. Chain APIs error misleading. Contributed by Harsh J Chouraria.

Modified:
    hadoop/mapreduce/trunk/CHANGES.txt
    
hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java

Modified: hadoop/mapreduce/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/CHANGES.txt?rev=1077761&r1=1077760&r2=1077761&view=diff
==============================================================================
--- hadoop/mapreduce/trunk/CHANGES.txt (original)
+++ hadoop/mapreduce/trunk/CHANGES.txt Fri Mar  4 04:52:35 2011
@@ -100,6 +100,9 @@ Trunk (unreleased changes)
     MAPREDUCE-2074. Task should fail when symlink creation fails.
     (Priyo Mustafi via tomwhite)
 
+    MAPREDUCE-1242. Chain APIs error misleading.
+    (Harsh J Chouraria via tomwhite)
+
 Release 0.22.0 - Unreleased
 
   INCOMPATIBLE CHANGES

Modified: 
hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java
URL: 
http://svn.apache.org/viewvc/hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java?rev=1077761&r1=1077760&r2=1077761&view=diff
==============================================================================
--- 
hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java
 (original)
+++ 
hadoop/mapreduce/trunk/src/java/org/apache/hadoop/mapreduce/lib/chain/Chain.java
 Fri Mar  4 04:52:35 2011
@@ -692,13 +692,13 @@ public class Chain {
           + CHAIN_MAPPER_CONFIG + (index - 1));
       if (!inputKeyClass.isAssignableFrom(previousMapperConf.getClass(
           MAPPER_OUTPUT_KEY_CLASS, null))) {
-        throw new IllegalArgumentException("The Mapper output key class does"
-            + " not match the previous Mapper input key class");
+        throw new IllegalArgumentException("The specified Mapper input key 
class does"
+            + " not match the previous Mapper's output key class.");
       }
       if (!inputValueClass.isAssignableFrom(previousMapperConf.getClass(
           MAPPER_OUTPUT_VALUE_CLASS, null))) {
-        throw new IllegalArgumentException("The Mapper output value class"
-            + " does not match the previous Mapper input value class");
+        throw new IllegalArgumentException("The specified Mapper input value 
class"
+            + " does not match the previous Mapper's output value class.");
       }
     }
   }