bryanck commented on pull request #3812: URL: https://github.com/apache/iceberg/pull/3812#issuecomment-1001789340
To give a little bit more detail, the SnapshotsTableScan and HistoryTableScan inherit from StaticTableScan to bypass the null snapshot check. However, during planning, `newRefinedSearch()` is called, which creates a StaticTableScan that does not override the null check. So this PR inherits `newRefinedSearch()` in the subclasses so they will instantiate the subclass (and bypass the null check). -- 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]
