GayathriSrividya commented on PR #3416: URL: https://github.com/apache/iceberg-python/pull/3416#issuecomment-4582444124
> Hi @rambleraptor and @Fokko — yes, PR #3416 was based on @jayceslesar's work from #2172. I've restructured the branch to credit them properly: > > * **Commit 1** (authored by @jayceslesar): Changes abstract method signatures and adds `iter()` wrapping for all non-REST catalogs (BigQuery, DynamoDB, Glue, Hive, NoOp, SQL) > * **Commit 2** (authored by me): Implements proper lazy pagination generators for the REST catalog with per-page `@retry`-decorated helpers > > All checks are passing on the latest push. Ready for review. Thanks! Hi @rambleraptor and @Fokko — yes, PR https://github.com/apache/iceberg-python/pull/3416 was based on @jayceslesar's work from https://github.com/apache/iceberg-python/pull/2172. I've restructured the branch to credit them properly: Commit 1 (authored by @jayceslesar): Changes abstract method signatures and adds iter() wrapping for all non-REST catalogs (BigQuery, DynamoDB, Glue, Hive, NoOp, SQL) Commit 2 (authored by me): Implements proper lazy pagination generators for the REST catalog with per-page @retry-decorated helpers All checks are passing on the latest push. Ready for review. Thanks! -- 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]
