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;
- }
-
}