nastra commented on code in PR #15368:
URL: https://github.com/apache/iceberg/pull/15368#discussion_r2827381528
##########
core/src/main/java/org/apache/iceberg/rest/RESTTable.java:
##########
@@ -75,11 +79,20 @@ public TableScan newScan() {
tableIdentifier,
resourcePaths,
supportedEndpoints,
- io(),
+ ioReference,
catalogProperties,
hadoopConf);
}
+ @Override
+ public FileIO io() {
Review Comment:
the main issue was where the FileIO in `RESTTableScan` would get storage
credentials from the server, but the FileIO instance here would still have the
table's IO without any storage credentials, which then causes permission
failures in real scenarios
--
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]
