eric-maynard opened a new pull request, #1576: URL: https://github.com/apache/polaris/pull/1576
Currently, many methods in `IcebergCatalogHandler` (formerly `IcebergCatalogHandlerWrapper`) fall back to implementations in `CatalogHandlers`, which sometimes can result in undesirable behavior. Two examples came up recently: 1. In #1285, we partially copied code out of `CatalogHandlers`, surfacing the fact that there's a hardcoded 4 retries on every commit 2. In #1528, it was [flagged](https://github.com/apache/polaris/pull/1528#discussion_r2080080529) that we could pass along pagination information to the catalog being delegated to rather than use the client-side pagination present in `CatalogHandlers` To move us away from `CatalogHandlers`, this PR introduces a new class `CatalogHandlerUtils` that's mostly a copy of `CatalogHandlers` as of Iceberg 1.8 and migrates usage of `CatalogHandlers` to this new type. Currently, the only meaningful change is that we inject a configuration store to make the number of retries on commit configurable. -- 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]
