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


   @StephanEwen , I have created and rebased with latest code several days ago. 
From my point of view, it seems a bit large change to involve 
`storage#shutdown`with `Dispatcher#cleanUpJobData`. My current implementation 
behaves like `CheckpointStore#shutdown` which would delete files asynchronously 
but not care about the returned future, and I think involving to 
`Dispatcher#cleanUpJobData` could be next stories. What do you think?


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