edgarRd opened a new pull request #1115: URL: https://github.com/apache/iceberg/pull/1115
This PR is a refactor of `TableScan` implementations to use a `TableScanContext` object for the optional arguments used. As shown in https://github.com/apache/iceberg/pull/1094#discussion_r435532764, when new arguments are added to a `TableScan` the number of changes needed is really large only to pass down the arguments through the refinement calls. This change will make such changes more straightforward minimizing the number of changes required to add an argument, which would only be required now in the context object and in the specific `TableScan` that would use it, leading to more maintainable code. PTAL @aokolnychyi @rdblue ---------------------------------------------------------------- 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]
