VigneshSK17 commented on code in PR #6099:
URL: https://github.com/apache/gravitino/pull/6099#discussion_r1903317182
##########
clients/cli/src/main/java/org/apache/gravitino/cli/GravitinoCommandLine.java:
##########
@@ -1192,6 +1192,13 @@ private void handleModelCommand() {
}
break;
+ case CommandActions.DELETE:
+ {
+ boolean force = line.hasOption(GravitinoOptions.FORCE);
+ newDeleteModel(url, ignore, force, metalake, catalog, schema,
model).handle();
+ break;
Review Comment:
I did update to fix this, but I'm curious if you mean that break should be
the last line of a case clause, even if there's braces around the code inside
the case clause?
--
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]
