jerryshao commented on code in PR #10661: URL: https://github.com/apache/gravitino/pull/10661#discussion_r3049811025
########## core/src/main/java/org/apache/gravitino/tag/TagManager.java: ########## Review Comment: No, I don't agree. The client SDK will get an undefined exception if we use `TagAlreadyExistsException`. This is not a REST protocol alone problem. This is a semantic change that should also align with the API/client implementation. Otherwise, the user will get a `TagAlreadyExistsException` that they don't expect to throw from the client call. -- 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]
