nastra commented on code in PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#discussion_r1812000392
##
aws/src/test/java/org/apache/iceberg/aws/s3/MinioUtil.java:
##
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contr
sdd commented on code in PR #665:
URL: https://github.com/apache/iceberg-rust/pull/665#discussion_r1811988261
##
crates/iceberg/src/spec/values.rs:
##
@@ -449,7 +456,30 @@ impl Datum {
PrimitiveLiteral::String(val) => ByteBuf::from(val.as_bytes()),
Prim
nastra commented on code in PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#discussion_r1811981848
##
aws/src/test/java/org/apache/iceberg/aws/s3/MinioUtil.java:
##
@@ -0,0 +1,64 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contr
ebyhr commented on PR #10999:
URL: https://github.com/apache/iceberg/pull/10999#issuecomment-2431050306
@Fokko Gentle reminder.
--
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 comme
nastra commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1811968596
##
open-api/src/testFixtures/java/org/apache/iceberg/rest/RESTCatalogServer.java:
##
@@ -64,7 +72,8 @@ public Map configuration() {
private CatalogContext initial
nastra commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1811965819
##
open-api/src/testFixtures/java/org/apache/iceberg/rest/RESTCatalogServer.java:
##
@@ -64,7 +72,9 @@ public Map configuration() {
private CatalogContext initial
sdd commented on PR #677:
URL: https://github.com/apache/iceberg-rust/pull/677#issuecomment-2431040483
This looks good to me, thanks! I'm assuming that the reason why someone
would want to perform a scan with an empty projection is to get a row count?
--
This is an automated message f
nastra commented on code in PR #11093:
URL: https://github.com/apache/iceberg/pull/11093#discussion_r1811964069
##
open-api/src/testFixtures/java/org/apache/iceberg/rest/RESTCatalogServer.java:
##
@@ -64,7 +65,9 @@ public Map configuration() {
private CatalogContext initial
sumedhsakdeo commented on code in PR #11130:
URL: https://github.com/apache/iceberg/pull/11130#discussion_r1811903184
##
format/spec.md:
##
@@ -298,16 +298,101 @@ Iceberg tables must not use field ids greater than
2147483447 (`Integer.MAX_VALU
The set of metadata columns is:
dramaticlly commented on PR #11377:
URL: https://github.com/apache/iceberg/pull/11377#issuecomment-2430994242
> LGTM ~
>
>
>
> @dramaticlly are you planning to add a doc update with this new option
sometime soon ?
Thank you @singhpk234 , that's next item on my list.
singhpk234 commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811926112
##
kafka-connect/kafka-connect/src/test/java/org/apache/iceberg/connect/data/RecordConverterTest.java:
##
@@ -206,11 +227,16 @@ public static void beforeAll() {
ajantha-bhat commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811920666
##
kafka-connect/kafka-connect/src/test/java/org/apache/iceberg/connect/data/RecordConverterTest.java:
##
@@ -206,11 +227,16 @@ public static void beforeAll() {
ajantha-bhat commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811920370
##
kafka-connect/kafka-connect/src/test/java/org/apache/iceberg/connect/data/RecordConverterTest.java:
##
@@ -206,11 +227,16 @@ public static void beforeAll() {
sullis commented on PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#issuecomment-2430833263
GitHub Actions CI passed.
--
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 com
advancedxy commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811794329
##
format/spec.md:
##
@@ -841,19 +855,45 @@ Notes:
## Delete Formats
-This section details how to encode row-level deletes in Iceberg delete files.
Row-level
amogh-jahagirdar commented on code in PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#discussion_r1811770900
##
core/src/main/java/org/apache/iceberg/ScanTasksIterable.java:
##
@@ -120,7 +146,21 @@ private void fetchScanTasks(String planTask) {
header
advancedxy commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811751380
##
format/puffin-spec.md:
##
@@ -123,6 +123,57 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct val
sullis commented on code in PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#discussion_r1811713157
##
aws/src/test/java/org/apache/iceberg/aws/s3/TestMinioUtil.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
sullis commented on code in PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#discussion_r1811711018
##
aws/src/test/java/org/apache/iceberg/aws/s3/TestS3FileIO.java:
##
@@ -88,12 +88,12 @@
import software.amazon.awssdk.services.s3.model.HeadObjectRequest;
import so
sullis commented on code in PR #11349:
URL: https://github.com/apache/iceberg/pull/11349#discussion_r1811706804
##
aws/src/test/java/org/apache/iceberg/aws/s3/TestMinioUtil.java:
##
@@ -0,0 +1,78 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more c
simonykq commented on issue #10710:
URL: https://github.com/apache/iceberg/issues/10710#issuecomment-2430696977
@pvary Hi, thanks for your reply.
Not sure if I understand this correctly, but if you want to write your own
serializer, I believe you need to do:
```
see.getConfig()
alex-kar commented on code in PR #179:
URL: https://github.com/apache/iceberg-go/pull/179#discussion_r1811640330
##
cmd/iceberg/main.go:
##
@@ -70,6 +71,7 @@ type Config struct {
Uuid bool `docopt:"uuid"`
Location bool `docopt:"location"`
Propsboo
github-actions[bot] commented on issue #10025:
URL: https://github.com/apache/iceberg/issues/10025#issuecomment-2430545401
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 occur
github-actions[bot] commented on issue #10044:
URL: https://github.com/apache/iceberg/issues/10044#issuecomment-2430545502
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 occur
github-actions[bot] commented on PR #9996:
URL: https://github.com/apache/iceberg/pull/9996#issuecomment-2430545302
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on issue #9966:
URL: https://github.com/apache/iceberg/issues/9966#issuecomment-2430545148
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 occurs.
github-actions[bot] commented on issue #10005:
URL: https://github.com/apache/iceberg/issues/10005#issuecomment-2430545354
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 occur
github-actions[bot] commented on issue #9990:
URL: https://github.com/apache/iceberg/issues/9990#issuecomment-2430545215
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 occurs.
github-actions[bot] commented on PR #10036:
URL: https://github.com/apache/iceberg/pull/10036#issuecomment-2430545454
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on issue #10029:
URL: https://github.com/apache/iceberg/issues/10029#issuecomment-2430545428
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 occur
github-actions[bot] commented on PR #10041:
URL: https://github.com/apache/iceberg/pull/10041#issuecomment-2430545471
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on PR #9957:
URL: https://github.com/apache/iceberg/pull/9957#issuecomment-2430545092
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on issue #10008:
URL: https://github.com/apache/iceberg/issues/10008#issuecomment-2430545372
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 occur
github-actions[bot] commented on PR #10002:
URL: https://github.com/apache/iceberg/pull/10002#issuecomment-2430545327
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pul
github-actions[bot] commented on PR #9993:
URL: https://github.com/apache/iceberg/pull/9993#issuecomment-2430545274
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on issue #9991:
URL: https://github.com/apache/iceberg/issues/9991#issuecomment-2430545239
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 occurs.
github-actions[bot] commented on PR #9968:
URL: https://github.com/apache/iceberg/pull/9968#issuecomment-2430545185
This pull request has been marked as stale due to 30 days of inactivity. It
will be closed in 1 week if no further activity occurs. If you think that’s
incorrect or this pull
github-actions[bot] commented on issue #9954:
URL: https://github.com/apache/iceberg/issues/9954#issuecomment-2430545073
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 occurs.
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811584252
##
format/spec.md:
##
@@ -841,19 +855,45 @@ Notes:
## Delete Formats
-This section details how to encode row-level deletes in Iceberg delete files.
Row-level del
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811583507
##
format/spec.md:
##
@@ -841,19 +855,45 @@ Notes:
## Delete Formats
-This section details how to encode row-level deletes in Iceberg delete files.
Row-level del
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811582684
##
format/spec.md:
##
@@ -51,6 +51,7 @@ Version 3 of the Iceberg spec extends data types and existing
metadata structure
* New data types: nanosecond timestamp(tz)
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811581790
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811581226
##
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 follo
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811581135
##
format/spec.md:
##
@@ -51,6 +51,7 @@ Version 3 of the Iceberg spec extends data types and existing
metadata structure
* New data types: nanosecond timestamp(tz)
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811580892
##
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 follo
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811580485
##
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 follo
rdblue commented on code in PR #11240:
URL: https://github.com/apache/iceberg/pull/11240#discussion_r1811571673
##
format/spec.md:
##
@@ -841,19 +855,45 @@ Notes:
## Delete Formats
-This section details how to encode row-level deletes in Iceberg delete files.
Row-level del
rdblue commented on PR #11130:
URL: https://github.com/apache/iceberg/pull/11130#issuecomment-2430515800
+1 to merge this since the vote has passed. We can do minor cleanup as we go
right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log o
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811569312
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811567559
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811564750
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811564153
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
rodmeneses commented on PR #11348:
URL: https://github.com/apache/iceberg/pull/11348#issuecomment-2430505790
> Do the schema conversions handle the nano ts correctly? Do we get a
`Timestamp(9)` when we convert to Flink schema, or a nano timestamp when we
convert from the Iceberg schema?
rdblue commented on code in PR #11238:
URL: https://github.com/apache/iceberg/pull/11238#discussion_r1811561158
##
format/puffin-spec.md:
##
@@ -123,6 +123,54 @@ The blob metadata for this blob may include following
properties:
- `ndv`: estimate of number of distinct values,
dramaticlly commented on PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#issuecomment-2430496861
Thanks everyone for the review and input, special thanks to @aokolnychyi for
optimized algorithm and @szehon-ho as original author and detailed review!
--
This is an automated messa
rahil-c commented on PR #11180:
URL: https://github.com/apache/iceberg/pull/11180#issuecomment-2430491796
@amogh-jahagirdar @rdblue @danielcweeks @nastra
Pushed the following commit `Utilize ParallelIterable, and use table prop
instead of rest planning`, which addressed some of the plann
natusioe commented on code in PR #173:
URL: https://github.com/apache/iceberg-go/pull/173#discussion_r1811534102
##
catalog/glue.go:
##
@@ -180,8 +267,155 @@ func (c *GlueCatalog) ListNamespaces(ctx context.Context,
parent table.Identifie
return icebergNamespaces, nil
szehon-ho commented on PR #9724:
URL: https://github.com/apache/iceberg/pull/9724#issuecomment-2430434532
Merged, thanks @dramaticlly !
--
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 specifi
szehon-ho merged PR #9724:
URL: https://github.com/apache/iceberg/pull/9724
--
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
ctagard opened a new issue, #11376:
URL: https://github.com/apache/iceberg/issues/11376
### Query engine
Spark
### Question
I was following the following tutorial
[here](https://github.com/databricks/docker-spark-iceberg) which uses the same
docker-compose as the Quicks
jinyangli34 commented on code in PR #11258:
URL: https://github.com/apache/iceberg/pull/11258#discussion_r1811499959
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewriteDataFilesAction.java:
##
@@ -557,8 +557,8 @@ public void testBinPackCombineMixedFile
jinyangli34 commented on code in PR #11258:
URL: https://github.com/apache/iceberg/pull/11258#discussion_r1811494439
##
parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java:
##
@@ -219,6 +222,52 @@ public void testTwoLevelList() throws IOException {
assertThat(
jinyangli34 commented on code in PR #11258:
URL: https://github.com/apache/iceberg/pull/11258#discussion_r1811493992
##
parquet/src/test/java/org/apache/iceberg/parquet/TestParquet.java:
##
@@ -219,6 +222,52 @@ public void testTwoLevelList() throws IOException {
assertThat(
jinyangli34 commented on PR #11258:
URL: https://github.com/apache/iceberg/pull/11258#issuecomment-2430386830
> > This makes it difficult to estimate the current row group size, and
result in creating much smaller row-group than
`write.parquet.row-group-size-bytes` config
>
> @jinyan
aihuaxu commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811485935
##
api/src/main/java/org/apache/iceberg/VariantLike.java:
##
@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribut
aihuaxu commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811485354
##
api/src/test/java/org/apache/iceberg/TestHelpers.java:
##
@@ -402,6 +406,98 @@ public int hashCode() {
}
}
+ /** A VariantLike implementation for testin
DieHertz commented on issue #1229:
URL:
https://github.com/apache/iceberg-python/issues/1229#issuecomment-2430363061
> If the work is done in Cython avro decoder -> pyarrow recordbatches using
PyArrow Cython API, then that also leaves room to release the GIL for
meaningful threaded concurr
aokolnychyi commented on PR #10935:
URL: https://github.com/apache/iceberg/pull/10935#issuecomment-2430250182
Coming back to some questions above.
> Do we want to resolve equality deletes and map them into data files? Or
should we add a new task and output the content of equality dele
corleyma commented on issue #1229:
URL:
https://github.com/apache/iceberg-python/issues/1229#issuecomment-2430215685
>IMO it makes sense to wait until it gets implemented or contribute there,
rather than writing our own general-purpose Avro-To-Arrow code.
We don't need general-purpos
amogh-jahagirdar commented on code in PR #11372:
URL: https://github.com/apache/iceberg/pull/11372#discussion_r1811329952
##
core/src/main/java/org/apache/iceberg/deletes/RoaringPositionBitmap.java:
##
@@ -0,0 +1,309 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) un
aihuaxu commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811347080
##
api/src/main/java/org/apache/iceberg/VariantLike.java:
##
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribut
singhpk234 commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811346546
##
kafka-connect/kafka-connect/src/test/java/org/apache/iceberg/connect/data/RecordConverterTest.java:
##
@@ -84,11 +93,18 @@
import org.apache.kafka.connect.stor
aihuaxu commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811345388
##
api/src/main/java/org/apache/iceberg/VariantLike.java:
##
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contribut
singhpk234 commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811341710
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -126,10 +127,14 @@ private Object convertValue(
cas
bryanck commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811332848
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -126,10 +127,14 @@ private Object convertValue(
case S
bryanck commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811331398
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -126,10 +127,14 @@ private Object convertValue(
case S
corleyma commented on issue #818:
URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2430157206
@shiv-io It should still be possible to do `load_view` _without_ supporting
any scanning functionality yet, and like @sungwy says, that is likely a
necessary precursor for othe
RussellSpitzer commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811314119
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -126,10 +127,14 @@ private Object convertValue(
bryanck commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811313113
##
kafka-connect/kafka-connect/src/test/java/org/apache/iceberg/connect/data/RecordConverterTest.java:
##
@@ -84,11 +93,18 @@
import org.apache.kafka.connect.storage
RussellSpitzer commented on code in PR #11346:
URL: https://github.com/apache/iceberg/pull/11346#discussion_r1811295504
##
kafka-connect/kafka-connect/src/main/java/org/apache/iceberg/connect/data/RecordConverter.java:
##
@@ -126,10 +127,14 @@ private Object convertValue(
zeroshade commented on code in PR #179:
URL: https://github.com/apache/iceberg-go/pull/179#discussion_r1811251550
##
cmd/iceberg/main.go:
##
@@ -70,6 +71,7 @@ type Config struct {
Uuid bool `docopt:"uuid"`
Location bool `docopt:"location"`
Propsbo
zeroshade commented on code in PR #177:
URL: https://github.com/apache/iceberg-go/pull/177#discussion_r1811229871
##
manifest.go:
##
@@ -567,6 +569,96 @@ func ReadManifestList(in io.Reader) ([]ManifestFile,
error) {
return out, dec.Error()
}
+// WriteManifestListV2 w
kevinjqliu commented on code in PR #11354:
URL: https://github.com/apache/iceberg/pull/11354#discussion_r1811248912
##
core/src/main/java/org/apache/iceberg/SnapshotParser.java:
##
@@ -140,6 +142,8 @@ static Snapshot fromJson(JsonNode node) {
}
}
summary =
zeroshade commented on PR #174:
URL: https://github.com/apache/iceberg-go/pull/174#issuecomment-2430012150
@Fokko @nastra rebased and updated, ready for review now!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
danielcweeks commented on code in PR #11368:
URL: https://github.com/apache/iceberg/pull/11368#discussion_r1811223257
##
api/src/main/java/org/apache/iceberg/UpdatePartitionSpec.java:
##
@@ -133,4 +133,16 @@ default UpdatePartitionSpec addNonDefaultSpec() {
throw new Unsupp
RussellSpitzer commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811183149
##
api/src/test/java/org/apache/iceberg/TestHelpers.java:
##
@@ -402,6 +406,98 @@ public int hashCode() {
}
}
+ /** A VariantLike implementation for
RussellSpitzer commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r1811179041
##
api/src/main/java/org/apache/iceberg/VariantLike.java:
##
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more co
stubz151 commented on code in PR #11021:
URL: https://github.com/apache/iceberg/pull/11021#discussion_r1811161067
##
aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIO.java:
##
@@ -443,6 +453,16 @@ public boolean recoverFile(String path) {
return recoverVersion.map(versio
jackye1995 commented on code in PR #11021:
URL: https://github.com/apache/iceberg/pull/11021#discussion_r1811146654
##
aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIO.java:
##
@@ -443,6 +453,16 @@ public boolean recoverFile(String path) {
return recoverVersion.map(vers
RussellSpitzer commented on code in PR #11324:
URL: https://github.com/apache/iceberg/pull/11324#discussion_r188395
##
api/src/main/java/org/apache/iceberg/VariantLike.java:
##
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more co
shiv-io commented on issue #818:
URL: https://github.com/apache/iceberg-python/issues/818#issuecomment-2429788539
I'm fairly new to the Iceberg ecosystem -- thanks for the insightful
discussion, looks like I have some reading to do before I can weigh in.
`load_view` aside though, I'd
zeroshade commented on PR #146:
URL: https://github.com/apache/iceberg-go/pull/146#issuecomment-2429784559
@jwtryg any updates?
--
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
zeroshade commented on code in PR #176:
URL: https://github.com/apache/iceberg-go/pull/176#discussion_r1811073199
##
io/gcs.go:
##
@@ -0,0 +1,63 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. See the NOTICE file
+//
RussellSpitzer commented on PR #11130:
URL: https://github.com/apache/iceberg/pull/11130#issuecomment-2429761860
@rdblue
@nastra
@sumedhsakdeo
@flyrain
@stevenzwu
@wgtmac
@aokolnychyi
@ashvina
@amogh-jahagirdar
Ping everyone, we've had the vote on the Mailing l
RussellSpitzer commented on code in PR #11130:
URL: https://github.com/apache/iceberg/pull/11130#discussion_r1811038409
##
format/spec.md:
##
@@ -298,16 +298,101 @@ Iceberg tables must not use field ids greater than
2147483447 (`Integer.MAX_VALU
The set of metadata columns i
rdblue commented on code in PR #11299:
URL: https://github.com/apache/iceberg/pull/11299#discussion_r1811023288
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/data/SparkPlannedAvroReader.java:
##
@@ -0,0 +1,190 @@
+/*
+ * Licensed to the Apache Software Foundation (A
rdblue merged PR #11299:
URL: https://github.com/apache/iceberg/pull/11299
--
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.ap
stubz151 commented on code in PR #11021:
URL: https://github.com/apache/iceberg/pull/11021#discussion_r1811021800
##
aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIO.java:
##
@@ -443,6 +453,16 @@ public boolean recoverFile(String path) {
return recoverVersion.map(versio
jackye1995 commented on code in PR #11021:
URL: https://github.com/apache/iceberg/pull/11021#discussion_r1810998409
##
aws/src/main/java/org/apache/iceberg/aws/s3/S3FileIOProperties.java:
##
@@ -837,6 +852,16 @@ public long s3RetryTotalWaitMs() {
return (long) s3RetryNumRet
aokolnychyi commented on code in PR #11299:
URL: https://github.com/apache/iceberg/pull/11299#discussion_r1811003876
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/data/SparkPlannedAvroReader.java:
##
@@ -0,0 +1,190 @@
+/*
+ * Licensed to the Apache Software Foundati
1 - 100 of 145 matches
Mail list logo
