rdblue commented on issue #15: Allow FileSystem provision to be overridden. URL: https://github.com/apache/incubator-iceberg/pull/15#issuecomment-442174208 This looks good to me, but I wonder: should this be done along with the addition of a FileIO API? Maybe you should supply the FileIO API and use that in HadoopTableOperations to avoid the need for this. If we do it at the same time, that's a bit less API churn.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services
