TEOTEO520 commented on PR #6718: URL: https://github.com/apache/gravitino/pull/6718#issuecomment-2760681751
> In the meantime, I would suggest you to split this PR to several small PRs. > > > 1. Add ModelChange API interface for comment modification > > 2. Implement comment update logic in model catalog and relational database > > 3. Add REST endpoint to support comment modification > > 4. Add Java client for model comment update > > 5. Add Python client for model comment update > > 6. Add UT and IT to validate comment alteration functionality > > For example, 1 and 2 can be a PR, 3, 4 and 5 can be another PR. Overall, it would be nice to control the PR size and to make it easy to review. OK, I'll split them -- 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: commits-unsubscr...@gravitino.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
