[GitHub] [incubator-iceberg] aokolnychyi merged pull request #509: Remove flaky Hive table tests

2019-10-01 Thread GitBox
aokolnychyi merged pull request #509: Remove flaky Hive table tests URL: https://github.com/apache/incubator-iceberg/pull/509 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-iceberg] prodeezy commented on issue #506: Handling Date type conversion as epochDay integer ordinal

2019-10-01 Thread GitBox
prodeezy commented on issue #506: Handling Date type conversion as epochDay integer ordinal URL: https://github.com/apache/incubator-iceberg/pull/506#issuecomment-536901662 Thanks @rdblue! This is an automated message from

[GitHub] [incubator-iceberg] mykolasmith closed issue #508: java.lang.IllegalStateException: Already closed file for partition

2019-10-01 Thread GitBox
mykolasmith closed issue #508: java.lang.IllegalStateException: Already closed file for partition URL: https://github.com/apache/incubator-iceberg/issues/508 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-iceberg] mykolasmith commented on issue #508: java.lang.IllegalStateException: Already closed file for partition

2019-10-01 Thread GitBox
mykolasmith commented on issue #508: java.lang.IllegalStateException: Already closed file for partition URL: https://github.com/apache/incubator-iceberg/issues/508#issuecomment-537083082 Thanks, solved. This is an automated

[GitHub] [incubator-iceberg] rdblue commented on issue #509: Remove flaky Hive table tests

2019-10-01 Thread GitBox
rdblue commented on issue #509: Remove flaky Hive table tests URL: https://github.com/apache/incubator-iceberg/pull/509#issuecomment-537106532 > do we think CATALOG_CACHE in HiveCatalogs is related to the problem with these tests? Yes! I forgot about that one, but it puts us over

[GitHub] [incubator-iceberg] rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330302946 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java

[GitHub] [incubator-iceberg] rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330357554 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java

[GitHub] [incubator-iceberg] rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330357554 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java

[GitHub] [incubator-iceberg] rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330287265 ## File path:

[GitHub] [incubator-iceberg] rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdsr commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330305047 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java

[GitHub] [incubator-iceberg] rdblue commented on a change in pull request #497: Support retaining last N snapshots

2019-10-01 Thread GitBox
rdblue commented on a change in pull request #497: Support retaining last N snapshots URL: https://github.com/apache/incubator-iceberg/pull/497#discussion_r330158860 ## File path: core/src/main/java/org/apache/iceberg/RemoveSnapshots.java ## @@ -82,6 +83,18 @@ public

[GitHub] [incubator-iceberg] rdblue commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdblue commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330279824 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java

[GitHub] [incubator-iceberg] rdblue commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40

2019-10-01 Thread GitBox
rdblue commented on a change in pull request #207: Add external schema mappings for files written with name-based schemas #40 URL: https://github.com/apache/incubator-iceberg/pull/207#discussion_r330283342 ## File path: core/src/main/java/org/apache/iceberg/avro/PruneColumns.java