pandeysambhi commented on issue #8037: URL: https://github.com/apache/gravitino/issues/8037#issuecomment-3179050996
@justinmclean I had the same thought while I was working on #7975 . I assumed this might have been implemented the way it is to support checking multiple fields at once. Whatever implementation of the validateFieldExistence I checked, the fieldName argument was being sent as a String array. Can you guide a little on how you want this to be improved, as if we go by throwing an exception, we would have to change the usage everywhere else as well. We can go for the nested field check as a simpler way, but I would want to understand the implications of the same. Let me know your thoughts, and I can start working on this. -- 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]
