curcur commented on pull request #13880: URL: https://github.com/apache/flink/pull/13880#issuecomment-722498104
Hey @tillrohrmann , thank you so much for reviewing the code! I believe I’ve addressed most of your comments except the “legacy scheduler strategy” problem. I will read the code of the new regional strategy tomorrow. If it turns out non-trivial of the change, what we are going to do with it… It is probably not very safe to make non-trivial changes on that critical piece just before the freeze date, especially I am not familiar with that part of the code? But no matter what, I would take a look at the code and evaluate the change first. ---------------------------------------------------------------- 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]
