rahil-c commented on PR #9695: URL: https://github.com/apache/iceberg/pull/9695#issuecomment-2033006567
@nastra @rdblue @danielcweeks @jackye1995 @amogh-jahagirdar When looking again at the `capabalities` pr: https://github.com/apache/iceberg/pull/9940, are we sure we want to add scan-planning as part of the `capabilities` list in the `ConfigResponse`? One question i want to raise is if server returns back `scan-planning` in the `capabalities`, would this mean that all tables under `RestCatalog` would support `scan planning`? I believe that we need a way tell clients the rest scan planning is supported, but I think this is something that should be done at the table level, in the `LoadTableResponse` as a new property? Let me know what you all think -- 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
