summarizer.setConf(conf) should be removed.

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

summarizer.setConf(conf) should be removed.

Stefan Groschupf-2
Hi,
getExtensionInstance() already set the conf in case it the class  
implements Configurable.
...
   if(object instanceof Configurable) {
             ((Configurable)object).setConf(this.conf);

.. so calling  summarizer.setConf(conf); sets the configuration a  
second time, what is useless.

Should I file a bug?

Stefan