tengqm commented on code in PR #6091:
URL: https://github.com/apache/gravitino/pull/6091#discussion_r1901682485
##########
clients/cli/src/main/java/org/apache/gravitino/cli/commands/Command.java:
##########
@@ -99,6 +100,16 @@ public static void setAuthenticationMode(String
authentication, String userName)
/** All commands have a handle method to handle and run the required
command. */
public abstract void handle();
+
+ /**
+ * verify the arguments. All commands have a verify method to verify the
arguments.
+ *
+ * @return Returns itself via argument validation, otherwise exits.
+ */
+ public Command verify() {
Review Comment:
I'd call this `validate` rather than `verify`.
`verify`, to me, is an action to confirm that something happened.
--
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]
