Abyss-lord opened a new pull request, #6104:
URL: https://github.com/apache/gravitino/pull/6104

   ### What changes were proposed in this pull request?
   
   Add `validate` method to Command, and refactor the validation code of 
catalog.
   
   ### Why are the changes needed?
   
   Fix: #6087 
   
   ### Does this PR introduce _any_ user-facing change?
   
   No
   
   ### How was this patch tested?
   
   ut + local test
   
   ```bash
   gcli catalog create -m demo_metalake --name test_catalog 
   # Missing --provider option.
   
   gcli catalog set -m demo_metalake --name Hive_catalog
   # Missing --property and --value options.
   
   gcli catalog set -m demo_metalake --name Hive_catalog --property propertyA
   # Missing --value option.
   
   gcli catalog set -m demo_metalake --name Hive_catalog --value valA
   # Missing --property option.
   
   gcli catalog remove -m demo_metalake --name Hive_catalog
   # Missing --property option.
   ```
   


-- 
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]

Reply via email to