amogh-jahagirdar opened a new pull request, #14480: URL: https://github.com/apache/iceberg/pull/14480
Co-authored-by: Prashant Singh <[email protected]> This change breaks away the server side changes from https://github.com/apache/iceberg/pull/13400 with the following differences: 1. Make RestCatalogAdapter more generically implement the scan planning APIs in a simple manner rather than have expectations on behavior based on the table name. This way servers using RESTCatalogAdapter can just use it as is, and this also helps other implementations just treat this as any other server to test against. plans are assigned an ID and stored in memory. At most one plan task will be returned, simulating using plan tasks as a mechanism for pagination. When a list of file scan tasks is put behind a plan task or when planning is performed async is defined by a provided PlanningBehavior interface. For unit tests in CatalogTests in #13400, we can override this PlanningBehavior interface so that we deterministically exercise the different paths, and can verify the client is robust to different cases. -- 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]
