RussellSpitzer commented on issue #16838: URL: https://github.com/apache/iceberg/issues/16838#issuecomment-4722727953
Sure, I don't think the NPE is really a big deal since it could only come from a custom catalog at the moment which is in violation of the specification. I know we generally write parsers and such so that they won't fail on bad input and instead we let the data go as far as it can until it hits an error like this. If you want to add a precondition though I don't think that's a huge deal. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
