Github user ramkrish86 commented on the issue:
https://github.com/apache/flink/pull/2409
If the lock scope is going to be moved for the entire triggerCheckpoint()
method then
`// since we released the lock in the meantime, we need to re-check
// that the conditions still hold. this is
clumsy, but it allows us to
// release the lock in the meantime while calls
to external services are
// blocking progress, and still gives us early
checks that skip work
// if no checkpoint can happen anyways`
Then the above comment and the checks that we perform below it can also be
removed since we don't release and acquire the lock again.
---
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.
---
