Github user bowenli86 commented on the issue:
https://github.com/apache/flink/pull/4764
I was commenting on the scenario described in `What is the purpose of the
change`. In that case, we want to either fail s2 or remove the lock for s2 as
you did. Since we are required to support parallel snapshots, the latter should
be the way to go.
LGTM ð ---
