kbendick opened a new pull request #2617: URL: https://github.com/apache/iceberg/pull/2617
Refines the error messages for metadata table scans that derive from `StaticTableScan` to be more user friendly, by putting the name of the metadata table in the error message This affects all scans of tables of most `MetadataTableType`s except for ` FilesTableScan` and `EntriesTableScan` (which could probably be refactored to also derive from `StaticTableScan` but currently derive from BaseTableScan directly). It also does not cover the aggregated ALL_* metadata table types, which derive rom `BaseAllMetadataTableScan` and for the most part implement an error message in the final derived classes themselves. This is to assist users who are testing incremental scans on a table `foo`, and might switch back and forth between querying `foo` and for example `foo.history` (to debug their development). Currently, the error message is just `Incremental scan is not supported`, which is not very clear when switching back and forth during development. This closes https://github.com/apache/iceberg/issues/2599 cc @RussellSpitzer @aokolnychyi -- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
