StephanEwen commented on pull request #15461: URL: https://github.com/apache/flink/pull/15461#issuecomment-811445749
Thanks for picking up these changes and taking the initiative to change the bookkeeping of splits, in addition to the bugfix. I originally meant to postpone the bookkeeping refactoring to a new PR, because we were still discussing about it and to safe time before the feature freeze. So, sorry to have caused extra work here. From my side, I think it looks fine at a first glance. Full review is below. But with the big changes here, I think this needs also a look from @becketqin. -- 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]
