Hi, I took a look at this issue as a learning exercise. The change was pretty trivial, but one unexpected benefit of the change was removing a minor, but potentially risky, runtime type-check from MetricsConsumerBolt.java.
_metricsConsumer.prepare(stormConf, _registrationArgument, context, (IErrorReporter)collector); _metricsConsumer.prepare(stormConf, _registrationArgument, context, collector); https://issues.apache.org/jira/browse/STORM-799 I would appreciate any feedback on this change. If the STORM-799 issue isn’t that significant, should be it be closed? Thanks, Aaron