ziqiangliang commented on PR #7392: URL: https://github.com/apache/gravitino/pull/7392#issuecomment-3073638401
> 1. I think it makes sense to delay the execution of multiple operations and eventually close the sql session, which can reduce people's mistakes during development. > 2. The necessity of methods like `callWithoutCommit` / `opWithoutCommit`. My question is that the `Operation` class has been defined, and it seems that the usage is to combine multiple `Operations` and finally use them in `doMultipleWithCommit`. Is it redundant to provide a separate method to build `Operation`? > 3. If it is determined that the new design method is sufficient to solve the problem, should the previous method be set to `deprecated` and prompt people to use the new method first? > 4. I see that there are currently multiple places where sessions are still opened and not closed(such as some methods in `TagMetaService`), and the current PR has not been completely fixed. Can all repairs be completed in the current PR first? @ziqiangliang Can you check it again? Thanks for the review. Regarding point 4, I will do a thorough check again on TagMetaService and other related call paths to ensure that all issues of sessions being opened without closing are fixed within this PR, to avoid any future misses. If I find that some of them are beyond the intended scope of this PR, I will fix what is within scope here and document the remaining ones for follow-up PRs, and update the team accordingly. -- 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]
