rkhachatryan commented on pull request #19331: URL: https://github.com/apache/flink/pull/19331#issuecomment-1086620964
Thanks for the review @Myasuka , > I think current solution looks a bit friable as it involes many changes to different APIs. I'm not sure what exactly do you mean. Indeed, there are changes to multiple interfaces, mostly because of the need to pass `RestoreMode` from `CheckpointRecoveryFactory` to `SharedStateRegistryFactory`. However, I think this is unavoidable. Or do you see any alternative? -- 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]
