PCloud63514 commented on issue #8203:
URL: https://github.com/apache/gravitino/issues/8203#issuecomment-3213056379
Thank you for the advice! I agree that the phrase "cheap insurance"
describes this well.
I’ll share this perspective with the reviewers to reinforce the need for
handling NPEs.
Fortunately, since NameIdentifier itself checks the name internally, it
seems we don’t need to provide any placeholder name or similar workaround
```java
private NameIdentifier(Namespace namespace, String name) {
check(namespace != null, "Cannot create a NameIdentifier with null
namespace");
check(
name != null && !name.isEmpty(), "Cannot create a NameIdentifier
with null or empty name");
this.namespace = namespace;
this.name = name;
}
```
--
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]
