thswlsqls commented on PR #16871: URL: https://github.com/apache/iceberg/pull/16871#issuecomment-5148439701
@pvary Thanks — you're right, the title claims the read is fixed when the change only replaces the NPE in `getPlainVectorAccessor` with a clear `UnsupportedOperationException`. I've renamed it to "Arrow: Throw a clear error instead of NPE when a vectorized read hits a null vector" and changed `Closes #10275` to `Related to #10275`, since fully reading added columns is still follow-up work there. Happy to use different wording if you'd prefer. -- 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]
