search

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
41 stars 13 forks source link

Fix ordering bug in DefaultRetryManager. #85

Closed Crim closed 6 years ago

Crim commented 6 years ago

Previously the implementation would return messageIds that were closest to the current timestamp instead of the messageIds which failed earlier. This means if there was a high rate of sustained failures, the earlier failed messageIds may never have gotten replayed.