GayathriSrividya commented on PR #3416: URL: https://github.com/apache/iceberg-python/pull/3416#issuecomment-4582445564
> @GayathriSrividya was this based off of #2172? If it was, we should cherry-pick in the commits so that the original author shows up in this PR as well. 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]
