[GitHub] rdblue commented on issue #15: Allow FileSystem provision to be overridden.

2018-11-27 Thread GitBox
rdblue commented on issue #15: Allow FileSystem provision to be overridden. URL: https://github.com/apache/incubator-iceberg/pull/15#issuecomment-442183487 I don't think that `FileIO` should handle other operations like `rename`. We could add `exists` to `InputFile` and `OutputFile`.

[GitHub] rdblue commented on issue #15: Allow FileSystem provision to be overridden.

2018-11-27 Thread GitBox
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