@TisonKun Unfortunately, I'm not familiar with this part of the code base and don't feel comfortable with merging this. As I don't have the context for this PR, my opinion would be to close this without merging (as well as corresponding ticket) as the CLI is not a performance-critical part of the code while the change might easily introduce a bug. Let me reiterate that I mostly say this, because I'm not familiar with this code. If there is another committer that is, it might be obvious to merge this PR.
[ Full content available at: https://github.com/apache/flink/pull/6491 ] This message was relayed via gitbox.apache.org for [email protected]
