snuyanzin commented on PR #22834:
URL: https://github.com/apache/flink/pull/22834#issuecomment-1621590942

   I would say it is more or less ok
   the only thing is that i still feel it's better to make the third arg 
optional as it was mentioned in jira
   Since now it looks more concise and changes should not be drastic...
   WDYT?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to