[GitHub] [incubator-iceberg] jerryshao closed pull request #396: Support DS read/write without specifying namespace

jerryshao closed pull request #396: Support DS read/write without specifying namespace URL: https://github.com/apache/incubator-iceberg/pull/396 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] [incubator-iceberg] jerryshao commented on a change in pull request #396: Support DS read/write without specifying namespace

jerryshao commented on a change in pull request #396: Support DS read/write without specifying namespace URL: https://github.com/apache/incubator-iceberg/pull/396#discussion_r316511612 ## File path: spark/src/main/java/org/apache/iceberg/spark/source/IcebergSource.java ##

[GitHub] [incubator-iceberg] aokolnychyi commented on issue #398: Push down StringStartsWith in Spark IcebergSource

aokolnychyi commented on issue #398: Push down StringStartsWith in Spark IcebergSource URL: https://github.com/apache/incubator-iceberg/pull/398#issuecomment-523698632 @sujithjay @rdblue @moulimukherjee I would appreciate your input on this.

[GitHub] [incubator-iceberg] aokolnychyi commented on a change in pull request #398: Push down StringStartsWith in Spark IcebergSource

aokolnychyi commented on a change in pull request #398: Push down StringStartsWith in Spark IcebergSource URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r316453652 ## File path: api/src/main/java/org/apache/iceberg/expressions/InclusiveMetricsEvaluator.java #

[GitHub] [incubator-iceberg] aokolnychyi commented on a change in pull request #398: Push down StringStartsWith in Spark IcebergSource

aokolnychyi commented on a change in pull request #398: Push down StringStartsWith in Spark IcebergSource URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r316452823 ## File path: parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java

[GitHub] [incubator-iceberg] aokolnychyi opened a new pull request #398: Push down StringStartsWith in Spark IcebergSource

aokolnychyi opened a new pull request #398: Push down StringStartsWith in Spark IcebergSource URL: https://github.com/apache/incubator-iceberg/pull/398 This PR adds support for pushdown of `StringStartsWith` predicates in Spark and resolves #397. --

[GitHub] [incubator-iceberg] aokolnychyi commented on issue #380: Rename some implementations for consistency

aokolnychyi commented on issue #380: Rename some implementations for consistency URL: https://github.com/apache/incubator-iceberg/pull/380#issuecomment-523687886 Sounds good to me! This is an automated message from the Apache

[GitHub] [incubator-iceberg] aokolnychyi commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore

aokolnychyi commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore URL: https://github.com/apache/incubator-iceberg/issues/366#issuecomment-523686687 I'll follow up with a PR then --

[GitHub] [incubator-iceberg] rdblue commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore

rdblue commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore URL: https://github.com/apache/incubator-iceberg/issues/366#issuecomment-523686330 Sounds like a blocker to me.

[GitHub] [incubator-iceberg] rdblue commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore

rdblue commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore URL: https://github.com/apache/incubator-iceberg/issues/366#issuecomment-523686279 Yeah, definitely. Good catch. ---

[GitHub] [incubator-iceberg] aokolnychyi edited a comment on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore

aokolnychyi edited a comment on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore URL: https://github.com/apache/incubator-iceberg/issues/366#issuecomment-523684134 @rdblue @rdsr what do you guys think about having some time between check

[GitHub] [incubator-iceberg] aokolnychyi commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore

aokolnychyi commented on issue #366: Concurrent appends to the same table using HiveCatalog lead to deadlock in the Metastore URL: https://github.com/apache/incubator-iceberg/issues/366#issuecomment-523684134 @rdblue @rdsr what do you guys think about adding a timeout between check lock r

[GitHub] [incubator-iceberg] rdblue commented on issue #380: Rename some implementations for consistency

rdblue commented on issue #380: Rename some implementations for consistency URL: https://github.com/apache/incubator-iceberg/pull/380#issuecomment-523660836 If you don't mind, I'd like to get some of the other PRs that touch these files in first, since this could cause conflicts and it shou

[GitHub] [incubator-iceberg] rdblue commented on a change in pull request #362: Support create and replace transactions in Catalog

rdblue commented on a change in pull request #362: Support create and replace transactions in Catalog URL: https://github.com/apache/incubator-iceberg/pull/362#discussion_r316267556 ## File path: hive/src/test/java/org/apache/iceberg/hive/HiveCreateReplaceTableTest.java ##

[GitHub] [incubator-iceberg] aokolnychyi commented on a change in pull request #362: Support create and replace transactions in Catalog

aokolnychyi commented on a change in pull request #362: Support create and replace transactions in Catalog URL: https://github.com/apache/incubator-iceberg/pull/362#discussion_r316228984 ## File path: hive/src/test/java/org/apache/iceberg/hive/HiveCreateReplaceTableTest.java #

[GitHub] [incubator-iceberg] aokolnychyi commented on a change in pull request #362: Support create and replace transactions in Catalog

aokolnychyi commented on a change in pull request #362: Support create and replace transactions in Catalog URL: https://github.com/apache/incubator-iceberg/pull/362#discussion_r316210566 ## File path: hive/src/test/java/org/apache/iceberg/hive/HiveCreateReplaceTableTest.java #