ssandona commented on issue #13526:
URL: https://github.com/apache/iceberg/issues/13526#issuecomment-3064784548
The previous behavior was setting the distribution mode to none. Do you now
from which version Is this new behavior? Can you point the related PR?
I was actually looking for
kadai0308 commented on PR #13087:
URL: https://github.com/apache/iceberg/pull/13087#issuecomment-3064737732
@Fokko
Hi, I am not sure about what's the next step of this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub an
dentiny commented on issue #1500:
URL: https://github.com/apache/iceberg-rust/issues/1500#issuecomment-3064716118
> Sorry I missed a "can" here 🫣
Cool, thanks again for the quick and helpful answer! I will check s3 sdk
then.
I will check how hard it is to implement.
--
This
Xuanwo commented on issue #1500:
URL: https://github.com/apache/iceberg-rust/issues/1500#issuecomment-3064698540
Sorry I missed a "can" here 🫣
--
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
stevenzwu commented on code in PR #13491:
URL: https://github.com/apache/iceberg/pull/13491#discussion_r2202294312
##
docs/mkdocs.yml:
##
@@ -22,69 +22,79 @@ plugins:
nav:
- index.md
- - Tables:
-- branching.md
-- configuration.md
-- evolution.md
-- mainte
ajantha-bhat commented on PR #13532:
URL: https://github.com/apache/iceberg/pull/13532#issuecomment-3064613757
@szehon-ho: Thanks for the review. I have updated the document.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
manuzhang commented on issue #13526:
URL: https://github.com/apache/iceberg/issues/13526#issuecomment-3064481468
Yes, because we agree it was a bug to implicitly changed to 'none' without
setting distribution mode.
--
This is an automated message from the Apache Git Service.
To respond to
geruh commented on issue #2095:
URL:
https://github.com/apache/iceberg-python/issues/2095#issuecomment-3064474306
I was incorrect about this looks like the java implementation allows for
primitives that are referenced by structs. However the determine partitioning
in pyiceberg doesn't have
ForeverAngry commented on issue #2201:
URL:
https://github.com/apache/iceberg-python/issues/2201#issuecomment-3064460915
> ### Question
> For my use case, I have a daily cron job that batch process and append
data but I only want a single snapshot record after the whole process. I tried
sdd commented on PR #1496:
URL: https://github.com/apache/iceberg-rust/pull/1496#issuecomment-3064451392
I put something together off the back of my earlier comment and opened a
draft PR. Here's the relevant commit:
https://github.com/apache/iceberg-rust/pull/1502/commits/2a02e559a71c30ebd9
sdd opened a new pull request, #1502:
URL: https://github.com/apache/iceberg-rust/pull/1502
## Which issue does this PR close?
- Follows on from https://github.com/apache/iceberg-rust/issues/482
- builds on https://github.com/apache/iceberg-rust/pull/1486
- Related to https://git
ajantha-bhat commented on PR #13523:
URL: https://github.com/apache/iceberg/pull/13523#issuecomment-3064394746
> I guess not a exact clean backport as its using new spark 4.0 procedure
framework :) but late lgtm
Yes. I have mentioned that after a clean backport, adopted to 4.0 procedu
github-actions[bot] commented on PR #13298:
URL: https://github.com/apache/iceberg/pull/13298#issuecomment-3064360294
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 #13084:
URL: https://github.com/apache/iceberg/pull/13084#issuecomment-3064359817
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 #11950:
URL: https://github.com/apache/iceberg/issues/11950#issuecomment-3064359640
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
sdd commented on PR #1496:
URL: https://github.com/apache/iceberg-rust/pull/1496#issuecomment-3064213003
Thanks for this contribution - I can see that you've put a lot of work into
this and it is appreciated!
I do have some concerns with this approach though - I think that by
mimicki
sdd commented on code in PR #1496:
URL: https://github.com/apache/iceberg-rust/pull/1496#discussion_r2202042699
##
crates/iceberg/Cargo.toml:
##
@@ -90,6 +90,7 @@ typed-builder = { workspace = true }
url = { workspace = true }
uuid = { workspace = true }
zstd = { workspace =
amogh-jahagirdar merged PR #13535:
URL: https://github.com/apache/iceberg/pull/13535
--
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...@
sdd commented on code in PR #1496:
URL: https://github.com/apache/iceberg-rust/pull/1496#discussion_r2202035793
##
crates/iceberg/src/delete_file_index.rs:
##
Review Comment:
Makes sense. If I recall, when `delete_file_index.rs` was added, the scan
module didn't exist as a
amogh-jahagirdar commented on PR #13535:
URL: https://github.com/apache/iceberg/pull/13535#issuecomment-3064160154
Thanks @anoopj , and @nandorKollar for reviewing.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
sdd commented on PR #1486:
URL: https://github.com/apache/iceberg-rust/pull/1486#issuecomment-3064117170
@liurenjie1024 / @Xuanwo It would be great to get a review of this if you
could find the time. This fully streaming approach is much cleaner and easier
to follow, with better error handl
sdd commented on code in PR #1017:
URL: https://github.com/apache/iceberg-rust/pull/1017#discussion_r2201925656
##
crates/iceberg/src/arrow/caching_delete_file_loader.rs:
##
@@ -308,28 +319,233 @@ impl CachingDeleteFileLoader {
Ok(result)
}
-/// Parses record
gtrettenero opened a new pull request, #2202:
URL: https://github.com/apache/iceberg-python/pull/2202
# Rationale for this change
# Are these changes tested?
# Are there any user-facing changes?
--
This is an automated message from the Apache G
dentiny commented on issue #1500:
URL: https://github.com/apache/iceberg-rust/issues/1500#issuecomment-3063641648
> We add this support in gcs
Sorry for the dumb question, does that mean it's already supported but I'm
unaware of? 😢
--
This is an automated message from the Apach
colinmarc commented on PR #1501:
URL: https://github.com/apache/iceberg-rust/pull/1501#issuecomment-3063600973
Famous last words! The CI issues seem to mostly revolve around everything
needing `protoc` to build now (because of datafusion-substrait).
--
This is an automated message from t
anoopj opened a new pull request, #13535:
URL: https://github.com/apache/iceberg/pull/13535
This ensures that when a CompositeMetricsReporter is closed, all of its
underlying reporters are properly closed as well, preventing resource leaks.
--
This is an automated message from the Apache
jbewing opened a new pull request, #13534:
URL: https://github.com/apache/iceberg/pull/13534
This PR fixes a case where tables with current identical partitioning
structures where one table was created directly w/ this structure and the other
was created, had it's partitions evolved in a wa
nastra merged PR #13509:
URL: https://github.com/apache/iceberg/pull/13509
--
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
nastra merged PR #13507:
URL: https://github.com/apache/iceberg/pull/13507
--
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
szehon-ho commented on code in PR #13532:
URL: https://github.com/apache/iceberg/pull/13532#discussion_r2201602850
##
docs/docs/spark-procedures.md:
##
@@ -974,6 +974,38 @@ Collect statistics of the snapshot with id `snap1` of
table `my_table` for colum
CALL catalog_name.syste
colinmarc opened a new pull request, #1501:
URL: https://github.com/apache/iceberg-rust/pull/1501
This did not require very many changes!
--
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 speci
jayceslesar commented on PR #2090:
URL: https://github.com/apache/iceberg-python/pull/2090#issuecomment-3063355842
@kevinjqliu can you please trigger tests again? Just want to see everything
hopefully work
--
This is an automated message from the Apache Git Service.
To respond to the mess
snowman2 commented on issue #2018:
URL:
https://github.com/apache/iceberg-python/issues/2018#issuecomment-3063310830
> Just to confirm, is this the result you’re looking for?
Not quite. It has to be an aiobotocore session. I would like it to look like:
```python
catalog = lo
saumyapandey1998 commented on issue #13506:
URL: https://github.com/apache/iceberg/issues/13506#issuecomment-3063293040
Hi @ebyhr! I keep running into the following error while trying to build the
project:
`A problem occurred configuring root project 'iceberg'.
> Could not resolve all
Xuanwo commented on issue #1500:
URL: https://github.com/apache/iceberg-rust/issues/1500#issuecomment-3063292765
We add this support in gcs. BTW, you can just use those HMAC key to access
gcs in S3 XML API.
--
This is an automated message from the Apache Git Service.
To respond to the mes
dentiny opened a new issue, #1500:
URL: https://github.com/apache/iceberg-rust/issues/1500
### Is your feature request related to a problem or challenge?
duckdb leverages HMAC keys for authentication:
https://duckdb.org/docs/stable/guides/network_cloud_storage/gcs_import.html
I
ehsantn commented on PR #2167:
URL: https://github.com/apache/iceberg-python/pull/2167#issuecomment-3063211683
@kevinjqliu please advise on next steps. This PR looks ready to merge to me
and the flakiness of existing unit tests should be addressed separately (would
be happy to contribute if
aihuaxu commented on code in PR #13284:
URL: https://github.com/apache/iceberg/pull/13284#discussion_r2201422332
##
api/src/test/java/org/apache/iceberg/variants/TestSerializedObject.java:
##
@@ -18,7 +18,7 @@
*/
package org.apache.iceberg.variants;
-import static org.asser
matthias-Q commented on PR #2173:
URL: https://github.com/apache/iceberg-python/pull/2173#issuecomment-3063182750
@kevinjqliu now that #2007 is merged should I add UUID conversion here or in
a separate PR?
--
This is an automated message from the Apache Git Service.
To respond to the mess
szehon-ho commented on PR #13523:
URL: https://github.com/apache/iceberg/pull/13523#issuecomment-3063155979
I guess not a exact clean backport as its using new spark 4.0 procedure
framework :) but late lgtm
--
This is an automated message from the Apache Git Service.
To respond to the me
kumarpritam863 commented on PR #12859:
URL: https://github.com/apache/iceberg/pull/12859#issuecomment-3063071747
@bryanck for review.
--
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
kumarpritam863 commented on PR #12954:
URL: https://github.com/apache/iceberg/pull/12954#issuecomment-3063069564
@bryanck can we please review this?
--
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
lliangyu-lin opened a new pull request, #13533:
URL: https://github.com/apache/iceberg/pull/13533
### Description
* Add support to build and publish iceberg docker images with bundled spark,
iceberg spark runtime, minio, and rest catalog.
* It is bundled because Docker compose cannot b
ajantha-bhat opened a new pull request, #13532:
URL: https://github.com/apache/iceberg/pull/13532
Follow up from https://github.com/apache/iceberg/pull/13523
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abov
ssandona commented on issue #13526:
URL: https://github.com/apache/iceberg/issues/13526#issuecomment-3062973461
Agree that the absence of write.distribution-mode
does not mean it is none. What I'm trying to say here is that ALTER TABLE...
WRITE LOCALLY ORDERED BY ideally should update tha
rmoff commented on issue #10745:
URL: https://github.com/apache/iceberg/issues/10745#issuecomment-3062941117
ack - I've passed this on.
--
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 specif
sullis commented on PR #13527:
URL: https://github.com/apache/iceberg/pull/13527#issuecomment-3062931700
fyi: I mentioned S3 Analytics Accelerator in my NYC Meetup talk on July 10th
https://docs.google.com/presentation/d/1GgBWJwxP_rZLMt4Kixv697CCBO_7vm1VM9YENs2MwdA/edit
--
manuzhang commented on issue #13526:
URL: https://github.com/apache/iceberg/issues/13526#issuecomment-3062926421
@ssandona The absence of distribution mode in the SQL doesn't mean the
distribution mode is `none`. `ALTER TABLE... WRITE LOCALLY ORDERED BY` should
only change the sort order wh
mimaison opened a new pull request, #13531:
URL: https://github.com/apache/iceberg/pull/13531
Iceberg has a service loader manifest for `IcebergSinkConnector` but does
not have a manifest for the Kafka Connect transformations.
Transformations also need a manifest, otherwise by default, wi
manuzhang commented on issue #10745:
URL: https://github.com/apache/iceberg/issues/10745#issuecomment-3062849055
I downloaded the zip, but it only contained the iceberg-kafka-connector jar,
not the same as that built following
https://iceberg.apache.org/docs/nightly/kafka-connect/#installat
jbewing opened a new issue, #13530:
URL: https://github.com/apache/iceberg/issues/13530
### Apache Iceberg version
1.9.0
### Query engine
Spark
### Please describe the bug 🐞
### What
If partitions are evolved for an Iceberg table, there is a potential to
amogh-jahagirdar commented on code in PR #13466:
URL: https://github.com/apache/iceberg/pull/13466#discussion_r2201039545
##
spark/v4.0/spark-runtime/src/integration/java/org/apache/iceberg/spark/TestSmoke.java:
##
@@ -31,7 +31,7 @@
import org.junit.jupiter.api.extension.Extend
RussellSpitzer commented on code in PR #13520:
URL: https://github.com/apache/iceberg/pull/13520#discussion_r2200946079
##
site/docs/community.md:
##
@@ -38,37 +62,31 @@ Issues are tracked in GitHub:
[open-issues]: https://github.com/apache/iceberg/issues
[new-issue]: https://
astarrr opened a new issue, #13529:
URL: https://github.com/apache/iceberg/issues/13529
### Feature Request / Improvement
Hello,
I think this feature is missing.
In `BaseMetastoreTableOperations#refreshFromMetadataLocation(String)` there
is an hardcoded value for Iceberg Tas
RussellSpitzer commented on code in PR #13520:
URL: https://github.com/apache/iceberg/pull/13520#discussion_r2200939675
##
site/docs/community.md:
##
@@ -20,16 +20,40 @@ title: "Community"
# Welcome!
-Apache Iceberg tracks issues in GitHub and prefers to receive contributio
stubz151 commented on code in PR #13347:
URL: https://github.com/apache/iceberg/pull/13347#discussion_r2200933124
##
aws/src/integration/java/org/apache/iceberg/aws/s3/TestS3InputStream.java:
##
@@ -160,28 +187,36 @@ private void readAndCheckRanges(
.isEqualTo(Arrays.co
rmoff commented on issue #10745:
URL: https://github.com/apache/iceberg/issues/10745#issuecomment-3062618564
By coincidence, and through no effort of my own, the connector is now live:
https://www.confluent.io/hub/iceberg/iceberg-kafka-connect
--
This is an automated message from the Apac
raphaelauv commented on issue #13481:
URL: https://github.com/apache/iceberg/issues/13481#issuecomment-3062572183
that was also my firt idea but iceberg 1.8.1 is already using the version
1.12.0 of avro
https://github.com/apache/iceberg/blob/9ce0fcf0af7becf25ad9fc996c3bad2afdcfd33d/gradle/l
alessandro-nori opened a new issue, #13528:
URL: https://github.com/apache/iceberg/issues/13528
### Feature Request / Improvement
Currently, S3InputStream throws a NotFoundException when getObject is called
on a location that doesn’t exist. This behavior ensures that
BaseMetastoreTab
stubz151 opened a new pull request, #13527:
URL: https://github.com/apache/iceberg/pull/13527
### What Am I doing
We are turning on AAL to be the default stream for S3FileIo. You can read
more about our optimizations in our
[readme](https://github.com/awslabs/analytics-accelerator-s3?tab
amogh-jahagirdar commented on code in PR #13507:
URL: https://github.com/apache/iceberg/pull/13507#discussion_r2200847375
##
core/src/main/java/org/apache/iceberg/rest/RESTMetricsReporter.java:
##
@@ -51,15 +57,20 @@ public void report(MetricsReport report) {
return;
troy-curtis commented on issue #11412:
URL: https://github.com/apache/iceberg/issues/11412#issuecomment-3062458414
I was also encountering this issue, and I also ended up creating longer
lived status credentials up front in the Spark driver and then passing them
into the catalog configurati
anoopj commented on code in PR #13507:
URL: https://github.com/apache/iceberg/pull/13507#discussion_r2200805021
##
core/src/main/java/org/apache/iceberg/rest/RESTMetricsReporter.java:
##
@@ -51,15 +57,20 @@ public void report(MetricsReport report) {
return;
}
-
paolo-cristofanelli commented on issue #13481:
URL: https://github.com/apache/iceberg/issues/13481#issuecomment-3062177184
Could it be that kafka-connect-avro-converter 7.9.2 uses avro-11.4 ( I am
looking
[here](https://mvnrepository.com/artifact/io.confluent/kafka-connect-avro-converter/7.
mwa28 commented on issue #2166:
URL:
https://github.com/apache/iceberg-python/issues/2166#issuecomment-3062014422
> Hi [@mwa28](https://github.com/mwa28), have you started working on this?
It would be great to get it in before the next release. If not, I can take it
up.
Hello @geruh
gaborkaszab commented on code in PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#discussion_r2200345749
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/actions/ExpireSnapshotsSparkAction.java:
##
@@ -79,6 +79,7 @@ public class ExpireSnapshotsSparkAction e
stubz151 commented on code in PR #13387:
URL: https://github.com/apache/iceberg/pull/13387#discussion_r2200312436
##
aws/src/integration/java/org/apache/iceberg/aws/s3/TestFlakyS3InputStream.java:
##
@@ -138,6 +230,14 @@ private S3ClientWrapper flakyStreamClient(AtomicInteger
c
sclee01 commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061627475
Great. I will take a look at the catalog side based on your point.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
ssandona opened a new issue, #13526:
URL: https://github.com/apache/iceberg/issues/13526
### Apache Iceberg version
1.7.1
### Query engine
Spark
### Please describe the bug 🐞
When running a `ALTER TABLE... WRITE LOCALLY ORDERED BY` statement it does
not set
DerGut commented on code in PR #1496:
URL: https://github.com/apache/iceberg-rust/pull/1496#discussion_r2200246689
##
crates/iceberg/src/metrics.rs:
##
@@ -0,0 +1,154 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor license agreements. S
phil-schreiber commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061491338
I just dove through the table creation and it seems the given IDs are sent
to the catalog rest endpoint, but it returns with the IDs overwritten by
sequential number. I thin
jiqiujia commented on issue #10181:
URL: https://github.com/apache/iceberg/issues/10181#issuecomment-306188
> From my side solved with use-table-distribution-and-ordering=false. Thank
you very much [@eubnara](https://github.com/eubnara)
@lrpt Are there any documentation illustrati
nastra commented on code in PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#discussion_r2200150993
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/actions/ExpireSnapshotsSparkAction.java:
##
@@ -79,6 +79,7 @@ public class ExpireSnapshotsSparkAction extend
manuzhang commented on PR #13521:
URL: https://github.com/apache/iceberg/pull/13521#issuecomment-3061366201
+1 to have a real page for REST spec under "Specification" tab.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
sclee01 commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061350212
I think somewhere in the java lib. I tested it using rest polaris catalog
and glue but they have same issue.
--
This is an automated message from the Apache Git Service.
To respo
rmoff commented on code in PR #13491:
URL: https://github.com/apache/iceberg/pull/13491#discussion_r2200128695
##
docs/mkdocs.yml:
##
@@ -22,69 +22,79 @@ plugins:
nav:
- index.md
- - Tables:
-- branching.md
-- configuration.md
-- evolution.md
-- maintenanc
rmoff commented on code in PR #13491:
URL: https://github.com/apache/iceberg/pull/13491#discussion_r2200125780
##
docs/mkdocs.yml:
##
@@ -22,69 +22,79 @@ plugins:
nav:
- index.md
- - Tables:
-- branching.md
-- configuration.md
-- evolution.md
-- maintenanc
phil-schreiber commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061312480
Awesome, I can confirm this is happening in tabluario rest catalog. Do you
have already any pointer, where this is going wrong? Is it on catalog side or
are the ids reset so
sclee01 commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061268323
@phil-schreiber Thanks for your comment. I will work on this and raise the
PR asap!
--
This is an automated message from the Apache Git Service.
To respond to the message, please
phil-schreiber commented on issue #13164:
URL: https://github.com/apache/iceberg/issues/13164#issuecomment-3061256178
Still running into this issue in 1.9.1. Any work on this would be much
appreciated. As @sclee01 mentioned for schema evolution to be viable it's
crucial to be able to specif
rmoff commented on code in PR #13512:
URL: https://github.com/apache/iceberg/pull/13512#discussion_r2200068278
##
site/docs/vendors.md:
##
@@ -61,6 +61,10 @@ the same copy of data using Spark and run analytics or AI
with our
[Machine Learning](https://www.cloudera.com/products
rmoff commented on code in PR #13511:
URL: https://github.com/apache/iceberg/pull/13511#discussion_r2200066519
##
site/docs/vendors.md:
##
@@ -132,10 +132,6 @@ The Stackable Data Platform is completely open source,
providing maximum portabi
Starburst is a commercial offering
keejon opened a new pull request, #13525:
URL: https://github.com/apache/iceberg/pull/13525
Fixes https://github.com/apache/iceberg/issues/13524
--
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 th
james5418 commented on issue #2018:
URL:
https://github.com/apache/iceberg-python/issues/2018#issuecomment-3061101620
Hi @snowman2, I’m interested in implementing this and will start with
**Option 1**!
Just to confirm, is this the result you’re looking for?
```python
my_sessio
gaborkaszab commented on PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#issuecomment-3061084396
Thanks for taking a look @nastra @amogh-jahagirdar !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
gaborkaszab commented on code in PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#discussion_r2199943583
##
spark/v4.0/spark/src/test/java/org/apache/iceberg/spark/actions/TestExpireSnapshotsAction.java:
##
@@ -1342,4 +1342,62 @@ public void testExpireSomeCheckFilesD
keejon opened a new issue, #13524:
URL: https://github.com/apache/iceberg/issues/13524
### Apache Iceberg version
1.9.1 (latest release)
### Query engine
Kafka Connect
### Please describe the bug 🐞
If `iceberg.catalog.client.region` is configured in connecto
gaborkaszab commented on code in PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#discussion_r2199849129
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/actions/ExpireSnapshotsSparkAction.java:
##
@@ -158,6 +165,10 @@ public Dataset expireFiles() {
gaborkaszab commented on code in PR #13509:
URL: https://github.com/apache/iceberg/pull/13509#discussion_r2199846533
##
spark/v4.0/spark/src/main/java/org/apache/iceberg/spark/actions/ExpireSnapshotsSparkAction.java:
##
@@ -79,6 +79,7 @@ public class ExpireSnapshotsSparkAction e
nastra merged PR #13523:
URL: https://github.com/apache/iceberg/pull/13523
--
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
90 matches
Mail list logo
