jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318403061
##
File path:
api/src/main/java/org/apache/iceberg/expressions/BoundSetPredicate.java
##
@@ -0,0 +1,176 @@
+/*
+
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402782
##
File path: api/src/test/java/org/apache/iceberg/expressions/TestEvaluator.java
##
@@ -362,4 +368,197 @@ public
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402818
##
File path: api/src/test/java/org/apache/iceberg/expressions/TestEvaluator.java
##
@@ -362,4 +368,197 @@ public
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402850
##
File path:
parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java
##
@@ -19,19 +19
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402737
##
File path: api/src/test/java/org/apache/iceberg/expressions/TestEvaluator.java
##
@@ -362,4 +368,197 @@ public
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402641
##
File path:
api/src/main/java/org/apache/iceberg/expressions/UnboundPredicate.java
##
@@ -125,13 +149,40 @@ pub
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402697
##
File path: api/src/test/java/org/apache/iceberg/expressions/TestEvaluator.java
##
@@ -362,4 +368,197 @@ public
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402469
##
File path:
api/src/main/java/org/apache/iceberg/expressions/BoundSetPredicate.java
##
@@ -0,0 +1,176 @@
+/*
+
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402583
##
File path:
api/src/main/java/org/apache/iceberg/expressions/UnboundPredicate.java
##
@@ -125,13 +149,40 @@ pub
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402556
##
File path: api/src/main/java/org/apache/iceberg/expressions/Predicate.java
##
@@ -39,37 +37,35 @@ public R ref(
jun-he commented on a change in pull request #357: Add in and not in predicates
URL: https://github.com/apache/incubator-iceberg/pull/357#discussion_r318402380
##
File path:
api/src/main/java/org/apache/iceberg/expressions/BoundSetPredicate.java
##
@@ -0,0 +1,176 @@
+/*
+
rdsr commented on a change in pull request #421: Add friendly names to catalog
tables
URL: https://github.com/apache/incubator-iceberg/pull/421#discussion_r318396277
##
File path: hive/src/main/java/org/apache/iceberg/hive/HiveCatalog.java
##
@@ -43,6 +43,11 @@ public Hive
chenjunjiedada commented on a change in pull request #374: Migrate spark table
to iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318368167
##
File path: spark/src/main/scala/org/apache/iceberg/spark/SparkTableUtil.scala
##
@@ -297,5 +30
rdblue commented on issue #416: Adding docs on how to use custom catalog with
iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#issuecomment-525518658
Merged. Thanks @moulimukherjee!
This is an automated messa
rdblue merged pull request #416: Adding docs on how to use custom catalog with
iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416
This is an automated message from the Apache Git Service.
To respond to the mes
rdblue opened a new pull request #421: Add friendly names to catalog tables
URL: https://github.com/apache/incubator-iceberg/pull/421
This adds an abstract `name` method to `BaseMetastoreCatalog` and a new
method to generate full table names from a catalog.
rdblue merged pull request #386: Fix copies in partition data and field
summaries
URL: https://github.com/apache/incubator-iceberg/pull/386
This is an automated message from the Apache Git Service.
To respond to the message,
moulimukherjee commented on a change in pull request #416: Adding docs on how
to use custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318324317
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,139 @@
+# Custom Catalog
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318324221
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,139 @@
+# Custom Catalog Impleme
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318324025
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,139 @@
+# Custom Catalog Impleme
rdblue commented on issue #420: Add commit.manifest-merge.enabled table property
URL: https://github.com/apache/incubator-iceberg/pull/420#issuecomment-525503667
@bryanck, here's the PR to disable manifest merging.
This is an
rdblue opened a new pull request #420: Add commit.manifest-merge.enabled table
property
URL: https://github.com/apache/incubator-iceberg/pull/420
This property can disable manifest merging. This is intended to be used with
`RewriteManifests`. Instead of automatic merging, `RewriteManifests
rdblue commented on issue #419: Adding common checks in commit()
URL: https://github.com/apache/incubator-iceberg/pull/419#issuecomment-525498398
Merged. Thanks @moulimukherjee!
This is an automated message from the Apache Git
rdblue merged pull request #419: Adding common checks in commit()
URL: https://github.com/apache/incubator-iceberg/pull/419
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
rdblue commented on issue #419: Adding common checks in commit()
URL: https://github.com/apache/incubator-iceberg/pull/419#issuecomment-525481988
Looks good to me. I'll merge once tests are passing.
This is an automated messag
moulimukherjee commented on a change in pull request #416: Adding docs on how
to use custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318295030
##
File path: site/mkdocs.yml
##
@@ -52,6 +52,7 @@ nav:
- Quickstart: api-q
moulimukherjee commented on a change in pull request #416: Adding docs on how
to use custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318291645
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog
moulimukherjee opened a new pull request #419: Adding common checks in commit()
URL: https://github.com/apache/incubator-iceberg/pull/419
Adding common checks in commit()
From conversation at
https://github.com/apache/incubator-iceberg/pull/416/files#r318243051
r? @rdblue @aok
moulimukherjee commented on a change in pull request #416: Adding docs on how
to use custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318268122
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog
rdblue commented on issue #349: Remove unreferenced hash computations
URL: https://github.com/apache/incubator-iceberg/pull/349#issuecomment-525455644
+1
Thanks for fixing this @jbapple!
This is an automated message fro
rdblue merged pull request #349: Remove unreferenced hash computations
URL: https://github.com/apache/incubator-iceberg/pull/349
This is an automated message from the Apache Git Service.
To respond to the message, please log
rdblue edited a comment on issue #280: Add persistent IDs to partition fields
URL:
https://github.com/apache/incubator-iceberg/issues/280#issuecomment-525451795
> And also as TableMetadata knows how many fields are in partition, so can
maintain the nextIDValue as well.
The next part
rdblue commented on issue #280: Add persistent IDs to partition fields
URL:
https://github.com/apache/incubator-iceberg/issues/280#issuecomment-525451795
> And also as TableMetadata knows how many fields are in partition, so can
maintain the nextIDValue as well.
The next partition f
rdblue commented on a change in pull request #374: Migrate spark table to
iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318254975
##
File path:
spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTableUtil.java
##
@@ -116,4 +
rdblue commented on a change in pull request #374: Migrate spark table to
iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318254975
##
File path:
spark/src/test/java/org/apache/iceberg/spark/source/TestSparkTableUtil.java
##
@@ -116,4 +
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318252717
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog Impleme
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318244402
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog Impleme
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318243982
##
File path: site/mkdocs.yml
##
@@ -52,6 +52,7 @@ nav:
- Quickstart: api-quickstar
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318243051
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog Impleme
rdblue commented on a change in pull request #416: Adding docs on how to use
custom catalog with iceberg
URL: https://github.com/apache/incubator-iceberg/pull/416#discussion_r318243051
##
File path: site/docs/custom-catalog.md
##
@@ -0,0 +1,149 @@
+# Custom Catalog Impleme
manishmalhotrawork commented on issue #280: Add persistent IDs to partition
fields
URL:
https://github.com/apache/incubator-iceberg/issues/280#issuecomment-525415789
@rdblue thanks.
Let me take a stab on this, may be as you mentioned in description.
Passing initial value for the
rdblue commented on issue #411: Ignore unsupported partition transforms for
forward compatibility
URL: https://github.com/apache/incubator-iceberg/pull/411#issuecomment-525412584
@aokolnychyi, can you review this?
This is an
rdblue commented on a change in pull request #374: Migrate spark table to
iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318200212
##
File path: spark/src/main/scala/org/apache/iceberg/spark/SparkTableUtil.scala
##
@@ -297,5 +302,81 @@
rdblue merged pull request #388: Handle rollback in snapshot expiration
URL: https://github.com/apache/incubator-iceberg/pull/388
This is an automated message from the Apache Git Service.
To respond to the message, please log
rdblue commented on issue #398: Push down StringStartsWith in Spark
IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#issuecomment-525398380
+1
There were a couple of minor points, but I think we can fix those in a
follow-up.
--
rdblue merged pull request #398: Push down StringStartsWith in Spark
IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398
This is an automated message from the Apache Git Service.
To respond to the message
rdblue closed issue #397: Push down StringStartsWith in Spark IcebergSource
URL: https://github.com/apache/incubator-iceberg/issues/397
This is an automated message from the Apache Git Service.
To respond to the message, plea
rdblue commented on a change in pull request #398: Push down StringStartsWith
in Spark IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r318197341
##
File path:
parquet/src/test/java/org/apache/iceberg/parquet/TestDictionaryRowGroupFilter.java
###
rdblue commented on a change in pull request #398: Push down StringStartsWith
in Spark IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r318196009
##
File path:
parquet/src/main/java/org/apache/iceberg/parquet/ParquetMetricsRowGroupFilter.java
###
rdblue commented on a change in pull request #398: Push down StringStartsWith
in Spark IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r318191642
##
File path:
api/src/test/java/org/apache/iceberg/expressions/TestInclusiveMetricsEvaluator.java
##
rdblue commented on a change in pull request #398: Push down StringStartsWith
in Spark IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#discussion_r318178750
##
File path:
api/src/test/java/org/apache/iceberg/expressions/TestInclusiveMetricsEvaluator.java
##
chenjunjiedada commented on a change in pull request #374: Migrate spark table
to iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318176144
##
File path: spark/src/main/scala/org/apache/iceberg/spark/SparkTableUtil.scala
##
@@ -297,5 +30
chenjunjiedada commented on a change in pull request #374: Migrate spark table
to iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318174610
##
File path: spark/src/main/scala/org/apache/iceberg/spark/SparkTableUtil.scala
##
@@ -297,5 +30
chenjunjiedada commented on a change in pull request #374: Migrate spark table
to iceberg table
URL: https://github.com/apache/incubator-iceberg/pull/374#discussion_r318171088
##
File path: spark/src/main/scala/org/apache/iceberg/spark/SparkTableUtil.scala
##
@@ -297,5 +30
aokolnychyi commented on issue #398: Push down StringStartsWith in Spark
IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398#issuecomment-525331795
I extended the scope of this PR to also cover filtering of manifests.
aokolnychyi closed pull request #398: Push down StringStartsWith in Spark
IcebergSource
URL: https://github.com/apache/incubator-iceberg/pull/398
This is an automated message from the Apache Git Service.
To respond to the me
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.
--
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_r318107696
##
File path: spark/src/main/java/org/apache/iceberg/spark/SparkFilters.java
##
@@ -153
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_r317970914
##
File path:
api/src/test/java/org/apache/iceberg/expressions/TestInclusiveMetricsEvaluator.jav
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_r317964972
##
File path:
api/src/test/java/org/apache/iceberg/expressions/TestInclusiveMetricsEvaluator.jav
60 matches
Mail list logo