pandeysambhi commented on code in PR #8191:
URL: https://github.com/apache/gravitino/pull/8191#discussion_r2286908931
##########
clients/cli/src/main/java/org/apache/gravitino/cli/commands/RoleDetails.java:
##########
@@ -56,7 +56,7 @@ public void handle() {
} catch (NoSuchMetalakeException err) {
exitWithError(ErrorMessages.UNKNOWN_METALAKE);
} catch (NoSuchUserException err) {
Review Comment:
I thought it should be NoSuchRoleException, but saw that issue mentioned
only the message update. I should have asked to clarify though. In order to
improve this, should I keep what was there and add an additional block for
NoSuchRoleException or replace the user exception with role?
--
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]
