Github user suez1224 commented on the issue: https://github.com/apache/flink/pull/5896 @aljoscha that's a very good catch. I added another commit to make sure the call order is always correct (https://github.com/apache/flink/pull/5896/commits/1298b04ea80b9160ac7e4ae8a171f80112a95945). Could you please let me know what you think? I think we should have the unittest in place since this is the second time it breaks. I can create another PR for the refactoring commit.
---