singhpk234 commented on code in PR #14563:
URL: https://github.com/apache/iceberg/pull/14563#discussion_r2517634232
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3372,6 +3372,10 @@ components:
status:
$ref: '#/components/schemas/PlanStatus'
const: "completed"
+ storage-credentials:
Review Comment:
my understanding if the creds are present in the scan response we should not
use the creds that were vended in loadTable ? hence some literature around of
this cred should be the one used when server returns it irrespective of what
was vended from the server ?
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3372,6 +3372,10 @@ components:
status:
$ref: '#/components/schemas/PlanStatus'
const: "completed"
+ storage-credentials:
Review Comment:
my understanding if the creds are present in the scan response we should not
use the creds that were vended in loadTable ? hence some literature around of
these cred **_should_** be the one used when server returns it irrespective of
what was vended from the loadTable ?
--
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]
