1996fanrui commented on pull request #18941: URL: https://github.com/apache/flink/pull/18941#issuecomment-1055494202
> LGTM. What classes had a merge conflict?(maybe I should take a look at it more carefully) Hi @akalash , thanks for your review. There are the following differences compared to 1.15: 1. CheckpointCoordinatorBuilder of 1.14 don't have default checkpointStatsTracker, but we add checkpointStatsTracker callback after checkpoint failed, caused some NPE in CheckpointCoordinatorTest. So I add default checkpointStatsTracker for CheckpointCoordinatorBuilder. 2. There is argument `boolean initializeBaseLocations` in initializeCheckpointLocation method of 1.15, so there are some differences in initializeCheckpointLocation. -- 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]
