Github user mliesenberg commented on the pull request:
https://github.com/apache/flink/pull/1097#issuecomment-150490748
I think that might just boil down to the question whether or not there is a
common enough use case for one without the other.
If so, we could change it to one method for check and one for populate
which would untangle the code a bit as well.
If there is no use case for one without the other, I'd agree to get rid of
the check.
I am unfortunately a bit pressed for time at the moment, but hope to find
time to address the other comments during the weekend and.
---
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.
---
