Myasuka commented on pull request #17151:
URL: https://github.com/apache/flink/pull/17151#issuecomment-920708455


   @dawidwys I think @Jiayi-Liao 's PR is near to merge and you could then 
rebase on his changes to continue working on the  feature of manual checkpoint 
only for tests.
   For the suggestion that initializing the base paths lazily within the 
`initializeLocationForCheckpoint`, I don't think this is a good idea as this 
should not be done on worker side.  How about initializing the locations when 
you first trigger the manual checkpoint on coordinator side?


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