You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@avro.apache.org by cu...@apache.org on 2009/05/20 22:25:14 UTC

svn commit: r776831 - /hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java

Author: cutting
Date: Wed May 20 20:25:14 2009
New Revision: 776831

URL: http://svn.apache.org/viewvc?rev=776831&view=rev
Log:
Remove some redundant code.  Inherited methods are identical.

Modified:
    hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java

Modified: hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java
URL: http://svn.apache.org/viewvc/hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java?rev=776831&r1=776830&r2=776831&view=diff
==============================================================================
--- hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java (original)
+++ hadoop/avro/trunk/src/java/org/apache/avro/specific/SpecificDatumReader.java Wed May 20 20:25:14 2009
@@ -39,11 +39,6 @@
     super(root, packageName);
   }
 
-  protected Object newRecord(Object old, Schema schema) {
-    Class c = getClass(schema.getName());
-    return(c.isInstance(old) ? old : newInstance(c));
-  }
-
   protected void addField(Object record, String name, int position, Object o) {
     ((SpecificRecord)record).set(position, o);
   }
@@ -54,19 +49,4 @@
     ((SpecificRecord)record).set(position, null);
   }
 
-  private Map<String,Class> classCache = new ConcurrentHashMap<String,Class>();
-
-  private Class getClass(String name) {
-    Class c = classCache.get(name);
-    if (c == null) {
-      try {
-        c = Class.forName(packageName + name);
-        classCache.put(name, c);
-      } catch (ClassNotFoundException e) {
-        throw new AvroRuntimeException(e);
-      }
-    }
-    return c;
-  }
-
 }