AHeise commented on pull request #12575:
URL: https://github.com/apache/flink/pull/12575#issuecomment-644589990


   > I've skimmed over the code (I didn't do the full review). Mostly it LGTM. 
General remark is that first couple of comments are missing explanation what 
are they doing in the commit message. They look like maybe they could be kept 
as individual commits, but it's hard to understand them now with the context, 
which such commit message could provide.
   
   Thank you for your feedback. I expanded the commit messages.


----------------------------------------------------------------
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.

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


Reply via email to