Github user zentol commented on the pull request:
https://github.com/apache/flink/pull/1355#issuecomment-156869041
I'm not too fond of the new version either and feel like this code bit
doesn't really need more clarity, simply because it isn't a lot of code.
Using a counter at all doesn't really make sense to me, you aren't counting
anything effectively. a Boolean seems more intuitive. It is nit-picky but we
are going for clarity here after all :)
---
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.
---
