Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/1190#issuecomment-194341881
@arunmahadevan Thanks for doing this. I still have not had time to dig
into this feature in as much detail as I would like, but from what I have seen
so far the contract with users looks good, and it is an important feature to
have. I have one concern now about thread safety in the
AckTrackingOutputCollector, but beyond that it looks fine.
I will leave support for best effort state checkpointing without at least
once processing for a later time. Like I said before I made an assumption that
was wrong, and that was part of why I got confused. I don't see it as a must
have feature, just confusion on my part.
---
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.
---
