rahil-c commented on PR #11180: URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2406717923
@rdblue @danielcweeks @amogh-jahagirdar @nastra @jackye1995 @singhpk234 Added a new commit `Add support for scan planning apis in REST Catalog` which invokes the new apis and introduces some new classes regarding a `RESTTable` and `RESTTableScan`. Let me know what you guys think and will try to address the recent feedback from @amogh-jahagirdar (thank you for the consistent reviews!) Finally once we have worked most of the major implementation bits will look into adding tests on this pr. Would also appreciate if people can take a look at this spec pr as well https://github.com/apache/iceberg/pull/11156/files regarding a change to `loadTable` for planning-mode. -- 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
