[#5832] Minor: metalake error message #5843
Conversation
|
lgtm |
| @@ -74,6 +74,6 @@ public MetalakeCreateRequest(String name, String comment, Map<String, String> pr | |||
| @Override | |||
| public void validate() throws IllegalArgumentException { | |||
| Preconditions.checkArgument( | |||
| StringUtils.isNotBlank(name), "\"name\" field is required and cannot be empty"); | |||
| StringUtils.isNotBlank(name), "\"name\" field is required. Please provide a valid name for the Metalake. It cannot be empty or only whitespace."); | |||
There was a problem hiding this comment.
The issue with this change is that it changes the Java API, not the Gravitino CLI. In the Java API context, this error message is correct, in the CLI context, the message could be more helpful. The change needs to be made in the CLI code not in the Java API code.
|
hi @muhammed-mamun Thank you for your contributions. |
|
The build failed because the code is not correctly formatted. We use Spotless to ensure the code is formatted consistently. To fix this, you'll need to run |
muhammed-mamun
changed the title
|
@muhammed-mamun are you still working on this? |
Why are these changes necessary?
The original error message was unclear, which could confuse users when they encountered an issue with the 'name' field. Improving the error message helps to provide clearer guidance to users.
Related Issues
This PR addresses issue #5832 , which reported an unclear error message when the 'name' field is empty.
#5832