GayathriSrividya commented on PR #3454: URL: https://github.com/apache/iceberg-python/pull/3454#issuecomment-4619010240
> @stark256-spec, @GayathriSrividya is also working on the pagination in #3416. Could you please coordinate the work so that we don't end up duplicating efforts? Thanks for flagging this. PR #3416 was opened first and it overlaps with #3454 on the REST lazy pagination work for #3365. The main difference is approach: - #3416 moves the catalog list APIs toward iterator semantics across catalogs. - #3454 keeps list compatibility and adds a lazy PaginationList for REST. I’m happy to coordinate and avoid duplicate work. If maintainers prefer the backward-compatible PaginationList approach, I can narrow or rework #3416 accordingly. If the iterator-based API is preferred, then I think #3416 remains the better home for the change. Please let me know which direction you’d like us to converge on. -- 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]
