PCloud63514 commented on PR #8233: URL: https://github.com/apache/gravitino/pull/8233#issuecomment-3213076110
@FANNG1 I shared these thoughts with the issue author and asked for their advice. I found the phrase “cheap insurance against hard-to-debug NPEs” very meaningful. Also, since the NameIdentifier constructor internally checks the name, I believe it’s better to rely on the constructor’s responsibility for data integrity rather than causing an early NPE. -- 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]
