Github user kleinsch commented on the pull request:
https://github.com/apache/storm/pull/780#issuecomment-147531078
I wasn't sure if I could make a non-backwards compatible change to
SpoutConfig, but seems like a good idea to move the implementation-specific
code out of there.
Renamed to FailedMsgRetryManagerFactory and moved params for the
exponential backoff manager into the ExponentialBackoffMsgRetryManagerFactory.
This needs to stay a factory bc the SpoutConfig has to be serialized to be
sent out to the nodes, then construct the FailedMsgRetryManager per-node. I
didn't want to require that the FailedMsgRetryManager be serializable, so said
the Factory would be serializable but the RetryManager doesn't need to be.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
