added better comment, removed code that is never called

Jason Jackson · Jason Jackson · commit d4f6114b77d6 · 2012-08-28T00:00:12.000-07:00
diff --git a/src/jvm/backtype/storm/Config.java b/src/jvm/backtype/storm/Config.java
@@ -373,7 +373,10 @@ public class Config extends HashMap<String, Object> {
     public static String TOPOLOGY_KRYO_REGISTER = "topology.kryo.register";
 
     /**
-     * JTODO
+     * A list of classes that customize storm's kryo instance during start-up.
+     * Each listed class name must implement IKryoDecorator. During start-up the 
+     * listed class is instantiated with 0 arguments, then its 'decorate' method 
+     * is called with storm's kryo instance as the only argument.
      */
     public static String TOPOLOGY_KRYO_DECORATORS = "topology.kryo.decorators";
 
@@ -562,10 +565,6 @@ public void registerSerialization(Class klass, Class<? extends Serializer> seria
         getRegisteredSerializations().add(register);        
     }
 
-    public void registerKryoDecorator(Class klass) {
-        getRegisteredKryoDecorators().add(klass.getName());
-    }
-    
     public void setSkipMissingKryoRegistrations(boolean skip) {
         put(Config.TOPOLOGY_SKIP_MISSING_KRYO_REGISTRATIONS, skip);
     }
@@ -592,11 +591,4 @@ private List getRegisteredSerializations() {
         }
         return (List) get(Config.TOPOLOGY_KRYO_REGISTER);
     }
-
-    private List getRegisteredKryoDecorators() {
-        if(!containsKey(Config.TOPOLOGY_KRYO_DECORATORS)) {
-            put(Config.TOPOLOGY_KRYO_DECORATORS, new ArrayList());
-        }
-        return (List) get(Config.TOPOLOGY_KRYO_DECORATORS);
-    }
 }
