What I meant is: Why have you decided current condition is an issue? Have you came across any problems with the check as it is currently in master? Before even analyzing a PR we should have a clear picture what is the reason for a change to be able to properly prioritize an important changes from not so much, but still requiring an effort of reviewing, verifying etc.
As far as I understand this is an isolated, internal component and a similar check is done one layer upwards. That's why I would really like to understand the reason for this PR. [ Full content available at: https://github.com/apache/flink/pull/6632 ] This message was relayed via gitbox.apache.org for [email protected]
