salesforce / storm-dynamic-spout

A framework for building spouts for Apache Storm and a Kafka based spout for dynamically skipping messages to be processed later.
BSD 3-Clause "New" or "Revised" License
40 stars 13 forks source link

Remove metric from VirtualSpout API #67

Closed stanlemon closed 6 years ago

stanlemon commented 6 years ago

I want to keep the VirtualSpout (and really the DelegateSpout interface) as minimal as possible. Whenever we have a method that's public for stuff like this I think we need to re-evaluate it and consider dropping it if possible. Thus, this metric is being moved into the VirtualSpout and I am dropping it from the interface.

stanlemon commented 6 years ago

Closing in favor of #78