nastra commented on code in PR #14519:
URL: https://github.com/apache/iceberg/pull/14519#discussion_r2509385577
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -1192,6 +1192,13 @@ paths:
- Catalog API
summary: Load vended credentials for a table from the catalog
operationId: loadCredentials
+ parameters:
+ - name: planId
+ in: query
+ required: false
+ schema:
+ type: string
+ description: The plan ID that has been used for server-side scan
planning
Review Comment:
I don't think that we would need to do any wrapping of FileIO per planId as
of now. Servers could technically just use different storage locations and then
send a different storage prefix in the credentials. That way we wouldn't need
to change any code on the client side, since we already keep track of FileIO
per storage prefix
--
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]
