Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/6020
A quick general question: could you observe any performance impact from
calling the `status()` method in the loops. It looks like a native method and I
am not sure that it is inexpensive. Maybe the better idea is to only check
`isValid()` in the loops and check `status()` only once after the loop to
ensure that everything was well and complete. Maybe that is also the reason why
this is split into two methods in the first place. What do you think?
---
