Author: cdouglas
Date: Tue Jul  8 18:34:55 2008
New Revision: 675078

URL: http://svn.apache.org/viewvc?rev=675078&view=rev
Log:
HADOOP-3706. Fix a ClassLoader issue in the mapred.join Parser that
prevents it from loading user-specified InputFormats.
Contributed by Jingkei Ly.

Modified:
    hadoop/core/trunk/CHANGES.txt
    hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/join/Parser.java

Modified: hadoop/core/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/CHANGES.txt?rev=675078&r1=675077&r2=675078&view=diff
==============================================================================
--- hadoop/core/trunk/CHANGES.txt (original)
+++ hadoop/core/trunk/CHANGES.txt Tue Jul  8 18:34:55 2008
@@ -788,6 +788,10 @@
     HADOOP-3630. Fix NullPointerException in CompositeRecordReader from empty
     sources (cdouglas)
 
+    HADOOP-3706. Fix a ClassLoader issue in the mapred.join Parser that
+    prevents it from loading user-specified InputFormats.
+    (Jingkei Ly via cdouglas)
+
 Release 0.17.2 - Unreleased
 
   BUG FIXES

Modified: hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/join/Parser.java
URL: 
http://svn.apache.org/viewvc/hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/join/Parser.java?rev=675078&r1=675077&r2=675078&view=diff
==============================================================================
--- hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/join/Parser.java 
(original)
+++ hadoop/core/trunk/src/mapred/org/apache/hadoop/mapred/join/Parser.java Tue 
Jul  8 18:34:55 2008
@@ -267,8 +267,8 @@
         Token t = i.next();
         if (TType.COMMA.equals(t.getType())) {
           try {
-            inf = (InputFormat)ReflectionUtils.newInstance(
-                Class.forName(sb.toString()).asSubclass(InputFormat.class),
+               inf = (InputFormat)ReflectionUtils.newInstance(
+                               job.getClassByName(sb.toString()),
                 job);
           } catch (ClassNotFoundException e) {
             throw (IOException)new IOException().initCause(e);


Reply via email to