GitHub user askprasanna opened a pull request:
https://github.com/apache/storm/pull/2152
[STORM-2505] OffsetManager should account for topic voids when returning
the count of committed offsets
The logic currently does a simple subtraction between the first and last
offset in the committed range. If the commit spans topic voids then this would
be incorrect. This incorrect number would always be on the higher side and
impact the logic around maximum uncommitted offsets allowable in the spout.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/askprasanna/storm master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/2152.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #2152
----
commit 5ae01efe4d2bf34e84bc342b1caf09ae8ce8e516
Author: Prasanna Ranganathan <[email protected]>
Date: 2017-06-07T12:33:09Z
[STORM-2505] OffsetManager should account for topic voids when returning
the count of committed offsets
----
---
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.
---
