Abyss-lord commented on PR #6327: URL: https://github.com/apache/gravitino/pull/6327#issuecomment-2606262928
> It would be possible to extend PR #6343 to include full names in the method signatures. However, care must be taken to avoid making the implementation overly abstract. Striking the right balance between abstraction and user comprehension is always a trade-off. Showing explicitly what is required for each method does have advantages. @justinmclean I’ve finished updating the code. Please take a look at the PR again when you have time, I think context is a great idea, and I want to extend PR #6343 when it merge. -- 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
