RocMarshal commented on PR #2268: URL: https://github.com/apache/incubator-streampark/pull/2268#issuecomment-1416658960
> > @1996fanrui Thx so much for the comments. I'll update it after the comments discussed. > > Hi @RocMarshal , thanks for your quick feedback. > > From your feedback, I found two main problems: > > * You added a lot of code that is not currently needed in this PR, such as: status, end_time, execution_mode, cluster_id, etc. And we can add them if they're needed in the future. > * Modify the logic of `CheckpointProcessor` as little as possible. The code of the master branch determines whether checkpoint/savepoint needs to be stored in the database according to the cache, and I think it is enough for now. Your code will access to the database frequently, it isn't friendly to database. > > Overall, this PR should only contain logic to trigger savepoint and fetch savepoint list. @1996fanrui thx for the response, and I'll keep the `minimum-change` for the next update. -- 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]
