FANNG1 commented on PR #649:
URL: https://github.com/apache/iceberg-rust/pull/649#issuecomment-2408419178
thanks @sdd @a-agmon @Xuanwo for the reviewing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
Xuanwo closed issue #644: Support pushdown more datafusion exprs to Iceberg
URL: https://github.com/apache/iceberg-rust/issues/644
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment
Xuanwo merged PR #649:
URL: https://github.com/apache/iceberg-rust/pull/649
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.a
Xuanwo merged PR #355:
URL: https://github.com/apache/iceberg-rust/pull/355
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@iceberg.a
chenwyi2 commented on PR #6570:
URL: https://github.com/apache/iceberg/pull/6570#issuecomment-2408332791
"Minimally Hive 2 HMS client is needed to use HIVE-26882 based locking" why
we have to check Hive 2? Suppose if i am in hive 1 and i cherry pick
HIVE-26882, that will not be right?
--
manuzhang commented on PR #10231:
URL: https://github.com/apache/iceberg/pull/10231#issuecomment-2408331380
@nastra Do we want to move forward with this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
amogh-jahagirdar commented on code in PR #11261:
URL: https://github.com/apache/iceberg/pull/11261#discussion_r1797529748
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestDelete.java:
##
@@ -1401,6 +1401,22 @@ public void
testDeleteToCustomWa
kumarpritam863 commented on PR #11298:
URL: https://github.com/apache/iceberg/pull/11298#issuecomment-2408288420
@bryanck Sir can we please review this one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
stevenzwu commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797505656
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the fo
github-actions[bot] commented on issue #596:
URL: https://github.com/apache/iceberg-python/issues/596#issuecomment-2408259229
This issue has been automatically marked as stale because it has been open
for 180 days with no activity. It will be closed in next 14 days if no further
activity oc
github-actions[bot] commented on PR #7914:
URL: https://github.com/apache/iceberg/pull/7914#issuecomment-2408258018
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If y
github-actions[bot] closed pull request #7914: Use SupportsPrefixOperations for
Remove OrphanFile Procedure
URL: https://github.com/apache/iceberg/pull/7914
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to
github-actions[bot] commented on PR #7561:
URL: https://github.com/apache/iceberg/pull/7561#issuecomment-2408258007
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If y
github-actions[bot] closed pull request #5888: Core: Rollback compaction on
conflicts
URL: https://github.com/apache/iceberg/pull/5888
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific co
github-actions[bot] commented on issue #8932:
URL: https://github.com/apache/iceberg/issues/8932#issuecomment-2408258032
This issue has been closed because it has not received any activity in the
last 14 days since being marked as 'stale'
--
This is an automated message from the Apache Gi
github-actions[bot] closed issue #8932: Slow RewriteManifests due to Validation
of Manifest Entries
URL: https://github.com/apache/iceberg/issues/8932
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
github-actions[bot] closed pull request #7561: Core: Fix retry behavior for
Jdbc Client
URL: https://github.com/apache/iceberg/pull/7561
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific
github-actions[bot] closed pull request #7267: API: Add
ParquetUtils.getSplitOffsets that takes an InputFile
URL: https://github.com/apache/iceberg/pull/7267
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above t
github-actions[bot] commented on PR #7267:
URL: https://github.com/apache/iceberg/pull/7267#issuecomment-2408258000
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If y
github-actions[bot] commented on PR #7127:
URL: https://github.com/apache/iceberg/pull/7127#issuecomment-2408257986
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If y
github-actions[bot] commented on PR #5888:
URL: https://github.com/apache/iceberg/pull/5888#issuecomment-2408257978
This pull request has been closed due to lack of activity. This is not a
judgement on the merit of the PR in any way. It is just a way of keeping the PR
queue manageable. If y
github-actions[bot] closed pull request #7127: [Core][Spark] Improve
DeleteOrphanFiles action to return additional details of deleted orphan files
URL: https://github.com/apache/iceberg/pull/7127
--
This is an automated message from the Apache Git Service.
To respond to the message, please lo
stevenzwu commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797498850
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the fo
stevenzwu commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797498850
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the fo
wypoon commented on code in PR #11292:
URL: https://github.com/apache/iceberg/pull/11292#discussion_r1797487178
##
arrow/src/main/java/org/apache/iceberg/arrow/vectorized/parquet/VectorizedColumnIterator.java:
##
@@ -214,20 +214,6 @@ protected int nextBatchOf(
}
}
- p
wypoon commented on code in PR #11292:
URL: https://github.com/apache/iceberg/pull/11292#discussion_r1797487178
##
arrow/src/main/java/org/apache/iceberg/arrow/vectorized/parquet/VectorizedColumnIterator.java:
##
@@ -214,20 +214,6 @@ protected int nextBatchOf(
}
}
- p
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797486598
##
core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java:
##
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
wypoon commented on code in PR #11247:
URL: https://github.com/apache/iceberg/pull/11247#discussion_r1797485367
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/data/parquet/vectorized/TestParquetDictionaryEncodedVectorizedReads.java:
##
@@ -93,4 +125,64 @@ public void
wypoon commented on code in PR #11247:
URL: https://github.com/apache/iceberg/pull/11247#discussion_r1797485367
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/data/parquet/vectorized/TestParquetDictionaryEncodedVectorizedReads.java:
##
@@ -93,4 +125,64 @@ public void
dependabot[bot] commented on PR #1225:
URL: https://github.com/apache/iceberg-python/pull/1225#issuecomment-2408219432
Superseded by #1228.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the spe
dependabot[bot] closed pull request #1225: Bump getdaft from 0.3.2 to 0.3.6
URL: https://github.com/apache/iceberg-python/pull/1225
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific commen
dependabot[bot] opened a new pull request, #1228:
URL: https://github.com/apache/iceberg-python/pull/1228
Bumps [getdaft](https://github.com/Eventual-Inc/Daft) from 0.3.2 to 0.3.8.
Release notes
Sourced from https://github.com/Eventual-Inc/Daft/releases";>getdaft's
releases.
ookumuso commented on PR #2:
URL: https://github.com/apache/iceberg/pull/2#issuecomment-2408212343
@danielcweeks @jackye1995 Updated the change to divide entropy into dirs so
we follow the following format now:
partitioned-path=true:
/data/0100/0110/0010/10101001/key=val/
anuragmantri commented on PR #11304:
URL: https://github.com/apache/iceberg/pull/11304#issuecomment-2408184398
StructLikeSet has an overridden `equals()` method which compared the
classes. This will[ fail when we compare
against](https://github.com/apache/iceberg/blob/67dc9e58cd57d953726677
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797443724
##
core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponseParser.java:
##
@@ -0,0 +1,138 @@
+/*
+ * Licensed to the Apache Software Foundatio
arkadius commented on code in PR #11305:
URL: https://github.com/apache/iceberg/pull/11305#discussion_r1797442838
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergSinkBuilder.java:
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797441874
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +70,47 @@ protected static Object[][] parameters() {
}
tomtongue commented on PR #11262:
URL: https://github.com/apache/iceberg/pull/11262#issuecomment-2408149842
@nastra Thanks for looking into this. When I tried the change, the
`@Parameters` annotation in an inherited classe is basically used, so this
change can solve the error (for the `@Par
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797440932
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,55 @@ protected static Object[][] parameters() {
}
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797440072
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,55 @@ protected static Object[][] parameters() {
}
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797439934
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,55 @@ protected static Object[][] parameters() {
}
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797439776
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestMetadataTables.java:
##
@@ -740,6 +743,10 @@ private boolean partitionMatch(
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797439642
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/sql/TestAlterTable.java:
##
@@ -275,6 +278,9 @@ public void testAlterColumnPositionFirst() {
@Tes
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797439446
##
spark/v3.5/spark-extensions/src/test/java/org/apache/iceberg/spark/extensions/TestMetadataTables.java:
##
@@ -18,8 +18,11 @@
*/
package org.apache.iceberg.
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797438018
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/sql/TestAlterTable.java:
##
@@ -275,6 +278,11 @@ public void testAlterColumnPositionFirst() {
@Te
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797438855
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +70,47 @@ protected static Object[][] parameters() {
}
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797263955
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,55 @@ protected static Object[][] parameters() {
}
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797427317
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -105,6 +166,10 @@ public void before() {
spark.conf().set("spar
emkornfield commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797423412
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the
emkornfield commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797421408
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the
emkornfield commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797420599
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797409933
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSLocation.java:
##
@@ -53,19 +63,17 @@ class ADLSLocation {
ValidationException.check(matcher.matches()
emkornfield commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1797405966
##
format/spec.md:
##
@@ -454,35 +457,40 @@ The schema of a manifest file is a struct called
`manifest_entry` with the follo
`data_file` is a struct with the
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797405095
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSLocation.java:
##
@@ -53,19 +63,17 @@ class ADLSLocation {
ValidationException.check(matcher.matches()
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797404275
##
azure/src/test/java/org/apache/iceberg/azure/adlsv2/ADLSLocationTest.java:
##
@@ -38,11 +38,26 @@ public void testLocationParsing(String scheme) {
assertThat(lo
emkornfield commented on PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#issuecomment-2408091808
> I’m not sure it’s worth drawing a line in the sand over this particular
issue and I’d like to talk about it a bit more as a community before we merge
this. I don’t want to set a p
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797403156
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSLocation.java:
##
@@ -18,24 +18,34 @@
*/
package org.apache.iceberg.azure.adlsv2;
+import java.net.URI;
rodmeneses commented on code in PR #11305:
URL: https://github.com/apache/iceberg/pull/11305#discussion_r1797397663
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergSinkBuilder.java:
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (A
zeroshade commented on issue #63:
URL: https://github.com/apache/iceberg-go/issues/63#issuecomment-2408083564
Go ahead and put the PR up and we'll discuss and iterate! Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
rodmeneses commented on code in PR #11305:
URL: https://github.com/apache/iceberg/pull/11305#discussion_r1797396711
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergSinkBuilder.java:
##
@@ -0,0 +1,82 @@
+/*
+ * Licensed to the Apache Software Foundation (A
jhump commented on issue #63:
URL: https://github.com/apache/iceberg-go/issues/63#issuecomment-2408081566
@zeroshade, I've got a fork where I've implemented DropTable and added
CreateTable and UpdateTable, which have the most complicated API. Did you
already have ideas on what the API might
emkornfield commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1797391875
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct va
allilou commented on issue #974:
URL: https://github.com/apache/iceberg-python/issues/974#issuecomment-2408060903
> As a workaround, you can define the S3 configurations (endpoint, secret
access key, and access key ID) in the load_rest method.
>
> ```python
> catalog = load_rest(
arkadius commented on PR #11219:
URL: https://github.com/apache/iceberg/pull/11219#issuecomment-2408058098
> > > Hi @arkadius I have started working in backporting the RANGE
distribution to the IcebergSink. The unit tests in my code will benefit from
the new marker interface you are introdu
rdblue commented on PR #11299:
URL: https://github.com/apache/iceberg/pull/11299#issuecomment-2408058064
Here are the benchmark results:
```
## main
Benchmark Mode Cnt
Score Error Units
IcebergSourceFlatAvroDataRead
arkadius opened a new pull request, #11305:
URL: https://github.com/apache/iceberg/pull/11305
This PR extracts the concept of `IcebergSinkBuilder` interface from the
#11219. This interface will be used to avoid code duplication in tests and to
keep the interface of both implemenations of si
allilou commented on issue #1087:
URL:
https://github.com/apache/iceberg-python/issues/1087#issuecomment-2408049850
> I updated my docker-compose.yaml to use extra_hosts and it worked. Closing
this issue.
I'm facing the same error, can you please give a snippet how you add the
extra
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797363531
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSLocation.java:
##
@@ -53,19 +63,17 @@ class ADLSLocation {
ValidationException.check(matcher.matches()
mrcnc commented on code in PR #11294:
URL: https://github.com/apache/iceberg/pull/11294#discussion_r1797360429
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/ADLSLocation.java:
##
@@ -53,19 +63,17 @@ class ADLSLocation {
ValidationException.check(matcher.matches()
aokolnychyi commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1797354699
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct va
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797318587
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,47 @@ protected static Object[][] parameters() {
}
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797318171
##
core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java:
##
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797317407
##
core/src/main/java/org/apache/iceberg/ScanTasksIterable.java:
##
@@ -0,0 +1,129 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
anuragmantri commented on PR #11304:
URL: https://github.com/apache/iceberg/pull/11304#issuecomment-2407956628
`TestStructLikeMap#testKeyAndEntrySetEquality()` is failing. Investigating
if `equals()` method needs some changes.
--
This is an automated message from the Apache Git Service.
T
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797315352
##
core/src/main/java/org/apache/iceberg/RESTPlanningMode.java:
##
@@ -0,0 +1,47 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more con
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797313224
##
core/src/main/java/org/apache/iceberg/rest/requests/FetchScanTasksRequest.java:
##
@@ -0,0 +1,45 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797312608
##
core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java:
##
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797310997
##
core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java:
##
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797310350
##
core/src/main/java/org/apache/iceberg/rest/responses/PlanTableScanResponse.java:
##
@@ -0,0 +1,127 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797309737
##
core/src/main/java/org/apache/iceberg/rest/responses/FetchScanTasksResponse.java:
##
@@ -0,0 +1,96 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) unde
rodmeneses commented on PR #11219:
URL: https://github.com/apache/iceberg/pull/11219#issuecomment-2407917469
> > Hi @arkadius I have started working in backporting the RANGE
distribution to the IcebergSink. The unit tests in my code will benefit from
the new marker interface you are introdu
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797273923
##
core/src/main/java/org/apache/iceberg/rest/RESTFileScanTaskParser.java:
##
@@ -0,0 +1,109 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
aokolnychyi commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1797275300
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct va
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797270059
##
.palantir/revapi.yml:
##
@@ -1058,6 +1058,11 @@ acceptedBreaks:
new: "method void
org.apache.iceberg.encryption.PlaintextEncryptionManager::()"
just
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797271023
##
core/src/main/java/org/apache/iceberg/rest/RESTContentFileParser.java:
##
@@ -0,0 +1,250 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ *
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797270059
##
.palantir/revapi.yml:
##
@@ -1058,6 +1058,11 @@ acceptedBreaks:
new: "method void
org.apache.iceberg.encryption.PlaintextEncryptionManager::()"
just
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797263955
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -59,18 +71,55 @@ protected static Object[][] parameters() {
}
anuragmantri opened a new pull request, #11304:
URL: https://github.com/apache/iceberg/pull/11304
`StructLikeMap` and `CharSequenceMap` methods `keySet()`, `values()`,
`entrySet()` return collections which when modified by consumers will change
the underlying maps. We should instead return
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797263690
##
core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequestParser.java:
##
@@ -0,0 +1,124 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797263023
##
core/src/main/java/org/apache/iceberg/rest/requests/PlanTableScanRequest.java:
##
@@ -0,0 +1,162 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797165832
##
core/src/main/java/org/apache/iceberg/RESTTable.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
haizhou-zhao commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1797262749
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/TestBaseWithCatalog.java:
##
@@ -89,13 +138,37 @@ public static void dropWarehouse() throws IOExceptio
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797262570
##
core/src/main/java/org/apache/iceberg/PartitionData.java:
##
@@ -72,6 +72,17 @@ public PartitionData(Types.StructType partitionType) {
this.stringSchema = sch
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797261523
##
core/src/main/java/org/apache/iceberg/GenericDataFile.java:
##
@@ -26,7 +26,7 @@
import org.apache.iceberg.relocated.com.google.common.collect.ImmutableMap;
impo
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797261187
##
.palantir/revapi.yml:
##
@@ -1058,6 +1058,11 @@ acceptedBreaks:
new: "method void
org.apache.iceberg.encryption.PlaintextEncryptionManager::()"
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797258000
##
api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java:
##
@@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec,
Exp
rahil-c commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797258444
##
core/src/main/java/org/apache/iceberg/rest/responses/FetchPlanningResultResponse.java:
##
@@ -0,0 +1,114 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797258000
##
api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java:
##
@@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec,
Exp
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1797258000
##
api/src/main/java/org/apache/iceberg/expressions/ResidualEvaluator.java:
##
@@ -89,6 +89,12 @@ public static ResidualEvaluator of(PartitionSpec spec,
Exp
aokolnychyi commented on PR #10935:
URL: https://github.com/apache/iceberg/pull/10935#issuecomment-2407845436
Will start looking into this PR today and should be able to finish over the
weekend.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
1 - 100 of 163 matches
Mail list logo
