Abyss-lord commented on issue #5861: URL: https://github.com/apache/gravitino/issues/5861#issuecomment-2547873689
> We would need to throw an exception so that no code in handle is called. The exception message can give the reason why. hi, @justinmclean Adding the `verify` to Command I have a few concerns 1. The scope of the change is too large, we need modify all subclass commands. 2. The potential for excessive duplication in validation logic. 3. The need to wrap the `handle` method in a try/catch block each time it is invoked. @shaofengshi could you plz help to see? -- 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
