kbendick commented on pull request #2740: URL: https://github.com/apache/iceberg/pull/2740#issuecomment-870022841
Will current Iceberg not work for v2 files at all, for example if the columns all wind up with plain encoding (as a trivial example)? I agree that catching the NPE isn't the most elegant solution, but if there's a possibility that page v2 records can be read if they only use the existing encoding, that would be nice. Otherwise, explicitly failing upon the encounter of a v2 page would be the best way to start. Please let me know if I can help somehow @samarthjain -- 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]
