[GitHub] [incubator-iceberg] aokolnychyi commented on issue #351: Provide an API to modify records within files

2019-08-07 Thread GitBox
aokolnychyi commented on issue #351: Provide an API to modify records within files URL: https://github.com/apache/incubator-iceberg/pull/351#issuecomment-519237696 > Do we need to add this to the constructor? I think it should be possible to start an overwrite, then scan the same table

[GitHub] [incubator-iceberg] aokolnychyi commented on issue #351: Provide an API to modify records within files

2019-08-05 Thread GitBox
aokolnychyi commented on issue #351: Provide an API to modify records within files URL: https://github.com/apache/incubator-iceberg/pull/351#issuecomment-518408801 @johnclara @rdblue I gave it more thought and it probably makes sense to merge this with `OverwriteFiles` API to avoid the