danielcweeks commented on PR #9872: URL: https://github.com/apache/iceberg/pull/9872#issuecomment-1986884426
I caught up with @rahil-c and discussed the plan forward here and he offered to consolidate all of the feedback across this and the other PR. Seems like best approach is just to close these two and start with a new one to incorporate all of the feedback. Just to reiterate, the plan here is not to change anything about what was agreed upon in terms of the pagination behavior, but we looked at the description, there were some aspects that expectations that were not clearly called out. This one is a little more complicated since we're trying add this in a backward compatible way and therefore we need to be clear about both old and new clients/servers and what the expectation is in each case. Thanks @rahil-c for taking this up! -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
