liurenjie1024 opened a new pull request, #1390:
URL: https://github.com/apache/iceberg-rust/pull/1390
## Which issue does this PR close?
This reverts commit 8bb9a882c7e5f6cfd2a9762136e874421cd7fb59.
See https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915710134
liurenjie1024 commented on issue #1386:
URL: https://github.com/apache/iceberg-rust/issues/1386#issuecomment-2915785708
cc @CTTY
--
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 comm
liurenjie1024 commented on code in PR #1369:
URL: https://github.com/apache/iceberg-rust/pull/1369#discussion_r2111508289
##
crates/iceberg/src/expr/visitors/manifest_evaluator.rs:
##
@@ -154,9 +155,19 @@ impl BoundPredicateVisitor for ManifestFilterVisitor<'_> {
_predi
liurenjie1024 merged PR #1369:
URL: https://github.com/apache/iceberg-rust/pull/1369
--
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...@
liurenjie1024 commented on code in PR #1383:
URL: https://github.com/apache/iceberg-rust/pull/1383#discussion_r2111516594
##
crates/iceberg/src/error.rs:
##
@@ -134,6 +134,8 @@ pub struct Error {
source: Option,
backtrace: Backtrace,
+
+retryable: bool,
Review C
mxm commented on PR #13072:
URL: https://github.com/apache/iceberg/pull/13072#issuecomment-2915875085
What are the obstacles for completely getting rid of TableSchema?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use th
Xuanwo closed issue #1329: Feature request: support to update snapshot summary
URL: https://github.com/apache/iceberg-rust/issues/1329
--
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
pvary commented on code in PR #13130:
URL: https://github.com/apache/iceberg/pull/13130#discussion_r2110933234
##
hive-metastore/src/test/java/org/apache/iceberg/hive/TestHiveCatalog.java:
##
@@ -1209,11 +1208,4 @@ public void
testDatabaseLocationWithSlashInWarehouseDir() {
andreaschiappacasse commented on issue #10294:
URL: https://github.com/apache/iceberg/issues/10294#issuecomment-2915516882
+1 on the issue since is still a problem in our workloads. @nastra would you
think is it possible to re-open it? Even if trino solved the upstream issue, it
seems that
pvary commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r2111329707
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -107,9 +115,12 @@ public abstract class TestRemoveOrphanFiles
andreaschiappacasse opened a new issue, #10294:
URL: https://github.com/apache/iceberg/issues/10294
### Apache Iceberg version
None
### Query engine
Athena (engine v3)
### Please describe the bug 🐞
Hello everyone, today my team incurred in a very strange bug
liurenjie1024 commented on PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915710134
Hi, @Xuanwo @dentiny We need to revert this pr, the new added update doesn't
exist in openapi spec:
https://github.com/apache/iceberg/blob/cab0decbb0e32bf314039e30807eb033c5066
liurenjie1024 commented on PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915721862
Also I don't think we should allow user to udpate snapshot summaries alone,
it's dangerous.
--
This is an automated message from the Apache Git Service.
To respond to the mes
dentiny commented on PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915720658
> Hi, @Xuanwo @dentiny We need to revert this pr, the new added update
doesn't exist in openapi spec:
https://github.com/apache/iceberg/blob/cab0decbb0e32bf314039e30807eb033c50665d5/
dentiny commented on PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915729660
> Also I don't think we should allow user to udpate snapshot summaries
alone, it's dangerous.
HI @liurenjie1024, curious if you have other suggestions to update snapshot
summa
liurenjie1024 merged PR #1390:
URL: https://github.com/apache/iceberg-rust/pull/1390
--
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...@
psarna opened a new issue, #443:
URL: https://github.com/apache/iceberg-go/issues/443
### Apache Iceberg version
None
### Please describe the bug 🐞
When running the `TestMergeManifests` test with `-race` flag, a data report
is created (attached to the end of the message)
manuzhang commented on code in PR #13104:
URL: https://github.com/apache/iceberg/pull/13104#discussion_r2111095523
##
docs/docs/spark-queries.md:
##
@@ -301,6 +301,16 @@ SELECT * FROM prod.db.table.entries;
| -- | -- | -- | -- | -- | -- |
| 2 | 57897183625154 | 0 | 0 |
{"cont
liurenjie1024 opened a new issue, #1389:
URL: https://github.com/apache/iceberg-rust/issues/1389
(no comment)
--
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
psavalle opened a new issue, #2051:
URL: https://github.com/apache/iceberg-python/issues/2051
### Feature Request / Improvement
The Glue catalog implementation [always reloads the
table](https://github.com/apache/iceberg-python/blob/main/pyiceberg/catalog/glue.py#L505)
in `GlueCatalo
wgtmac commented on code in PR #107:
URL: https://github.com/apache/iceberg-cpp/pull/107#discussion_r2111250416
##
.github/workflows/sanitizer_test.yml:
##
@@ -0,0 +1,63 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.
raulcd commented on PR #107:
URL: https://github.com/apache/iceberg-cpp/pull/107#issuecomment-2915302693
I am slightly confused, the log output from the job, see the artifact
uploaded here:
https://github.com/apache/iceberg-cpp/actions/runs/15293677546?pr=107
suggest there are several
b-rick commented on code in PR #12424:
URL: https://github.com/apache/iceberg/pull/12424#discussion_r2111204802
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/dynamic/RowDataEvolver.java:
##
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software Foundation (
wgtmac opened a new pull request, #113:
URL: https://github.com/apache/iceberg-cpp/pull/113
(no comment)
--
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-m
liurenjie1024 commented on code in PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#discussion_r2111438158
##
crates/iceberg/src/catalog/mod.rs:
##
@@ -495,6 +495,12 @@ pub enum TableUpdate {
/// Schema IDs to remove.
schema_ids: Vec,
},
+
liurenjie1024 commented on PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336#issuecomment-2915765649
> > Also I don't think we should allow user to udpate snapshot summaries
alone, it's dangerous.
>
> HI @liurenjie1024, curious if you have other suggestions to update
sn
liurenjie1024 commented on PR #1390:
URL: https://github.com/apache/iceberg-rust/pull/1390#issuecomment-2915775716
cc @Xuanwo @sdd @Fokko @kevinjqliu
--
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
liurenjie1024 commented on issue #1382:
URL: https://github.com/apache/iceberg-rust/issues/1382#issuecomment-2915783715
> Hi [@liurenjie1024](https://github.com/liurenjie1024) , thanks for having
this!
>
> Would it be ok if I take up the commit path work? I can start working on a
POC
liurenjie1024 commented on issue #1382:
URL: https://github.com/apache/iceberg-rust/issues/1382#issuecomment-2915783358
> I don't get why we need this. Is this used by the catalog implementation,
e.g. SqlCatalog::update_table? Could you elaborate more the commit process and
where we need th
liziyan-lzy commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r237312
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/DeleteOrphanFilesSparkAction.java:
##
@@ -303,39 +310,89 @@ private Dataset listedFileDS() {
liziyan-lzy commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r237789
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -344,6 +374,8 @@ public void testWapFilesAreKept() {
jbonofre commented on PR #13136:
URL: https://github.com/apache/iceberg/pull/13136#issuecomment-2915277304
I will take a look as well. Thanks !
--
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
jbonofre commented on PR #13136:
URL: https://github.com/apache/iceberg/pull/13136#issuecomment-2915276780
@nastra @Fokko ^^
--
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.
liziyan-lzy commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r245839
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -107,9 +115,12 @@ public abstract class TestRemoveOrpha
mxm commented on code in PR #12424:
URL: https://github.com/apache/iceberg/pull/12424#discussion_r265456
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/dynamic/RowDataEvolver.java:
##
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111663422
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,29 @@ private static PartitionMap
computeStatsDiff(
Sets.newHas
Xuanwo merged PR #1336:
URL: https://github.com/apache/iceberg-rust/pull/1336
--
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
ajantha-bhat commented on issue #13155:
URL: https://github.com/apache/iceberg/issues/13155#issuecomment-2916012247
I found a clean way. Existing testacase was failing because I included same
manifest twice when reused. Now, I added a snapshot id check. So, each snapshot
only considers the
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111647311
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,29 @@ private static PartitionMap
computeStatsDiff(
Sets.newHas
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111640567
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,22 @@ private static PartitionMap
computeStatsDiff(
Sets.newHas
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111642709
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -276,7 +275,8 @@ private static Collection
computeAndMergeStatsIncremental(
Part
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111647311
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,29 @@ private static PartitionMap
computeStatsDiff(
Sets.newHas
JeonDaehong commented on PR #13165:
URL: https://github.com/apache/iceberg/pull/13165#issuecomment-2916040410
@nastra
Hello,
I’ve completed the backport.
Please review and merge it when you get a chance!
Also, this is a separate topic —
After my previous PR was mer
hasanozciftcii commented on issue #974:
URL: https://github.com/apache/iceberg-python/issues/974#issuecomment-2916096063
> As a workaround, you can define the S3 configurations (endpoint, secret
access key, and access key ID) in the load_rest method.
>
> catalog = load_rest(
>
pvary commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111741154
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,29 @@ private static PartitionMap
computeStatsDiff(
Sets.newHashSet(
fivetran-raulpersa opened a new pull request, #13169:
URL: https://github.com/apache/iceberg/pull/13169
Fixes #13014
In multiple Table ops (Glue, Dynamo, BigQuery and Nessie) a failing
operation will clean up the metadata file.
However it should only do so if it was the one to cre
pvary merged PR #13165:
URL: https://github.com/apache/iceberg/pull/13165
--
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.apa
dentiny opened a new pull request, #1391:
URL: https://github.com/apache/iceberg-rust/pull/1391
## Which issue does this PR close?
- Closes https://github.com/apache/iceberg-rust/issues/1329
## What changes are included in this PR?
This PR tries to do the same thing as re
zeroshade opened a new pull request, #445:
URL: https://github.com/apache/iceberg-go/pull/445
fixes #443
Eliminates potential race condition in scanning caused by the metrics
evaluator used for manifest pruning.
Since the stats are only used for a given run of Eval, we don't ne
lliangyu-lin opened a new pull request, #446:
URL: https://github.com/apache/iceberg-go/pull/446
### Description
When calling NewCatalog in glue catalog without any AWS properties, causing
the catalog properties to be nil
```
props: iceberg.Properties(glueOps.awsProperties),
Fokko merged PR #108:
URL: https://github.com/apache/iceberg-cpp/pull/108
--
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.apa
Fokko commented on PR #108:
URL: https://github.com/apache/iceberg-cpp/pull/108#issuecomment-2917339595
LGTM @dongxiao1198 , thanks for the review @wgtmac, @lidavidm and
@yingcai-cy
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
smaheshwar-pltr opened a new issue, #13179:
URL: https://github.com/apache/iceberg/issues/13179
### Feature Request / Improvement
Spark's [incremental append
reading](https://iceberg.apache.org/docs/latest/spark-queries/#incremental-read)
is useful for append-only incremental workflo
smaheshwar-pltr commented on PR #2031:
URL: https://github.com/apache/iceberg-python/pull/2031#issuecomment-2917350349
Put up https://github.com/apache/iceberg/issues/13179 re an append-only
option
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
smaheshwar-pltr commented on issue #13179:
URL: https://github.com/apache/iceberg/issues/13179#issuecomment-2917349624
Not sure who to ping here so CC-ing folks I see on previous Spark
incremental read work 😄: @aokolnychyi @huaxingao @szehon-ho @RussellSpitzer,
please let me know your thoug
ajantha-bhat opened a new issue, #13178:
URL: https://github.com/apache/iceberg/issues/13178
### Apache Iceberg version
None
### Query engine
None
### Please describe the bug 🐞
`./gradlew clean build -x test -x integrationTest --no-build-cache`
Defaul
dentiny commented on code in PR #1391:
URL: https://github.com/apache/iceberg-rust/pull/1391#discussion_r2112436407
##
Cargo.lock:
##
@@ -3712,7 +3712,7 @@ dependencies = [
"iceberg-catalog-rest",
"iceberg-datafusion",
"iceberg_test_utils",
- "ordered-float 4.6.0",
+ "orde
zeroshade merged PR #440:
URL: https://github.com/apache/iceberg-go/pull/440
--
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.
dentiny closed issue #1392: Feature request: expose `DataFile` builder as public
URL: https://github.com/apache/iceberg-rust/issues/1392
--
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 c
dentiny commented on issue #1392:
URL: https://github.com/apache/iceberg-rust/issues/1392#issuecomment-2917202154
Sorry I get fooled by gpt and deepseek, the default visibility is public
Ref: https://docs.rs/derive_builder/latest/derive_builder/#setter-visibility
--
This is an automated
TomasDarquier commented on issue #13178:
URL: https://github.com/apache/iceberg/issues/13178#issuecomment-2917209543
- [x] I can contribute a fix for this bug independently
Hi, I’d like to work on this issue — it seems like a great fit for my first
contribution.
--
This is an automate
ajantha-bhat commented on issue #13178:
URL: https://github.com/apache/iceberg/issues/13178#issuecomment-2917115167
Good first issue for new contributors.
--
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
dentiny opened a new issue, #1392:
URL: https://github.com/apache/iceberg-rust/issues/1392
### Is your feature request related to a problem or challenge?
My workflow looks like
- Accumulate arrow batches in-memory, flush to local filesystem when it
reaches certain threshold
- Pe
pvary commented on PR #13165:
URL: https://github.com/apache/iceberg/pull/13165#issuecomment-2916219167
Merged to main.
Thanks @JeonDaehong for the backport and @nastra for the review.
>After my previous PR was merged, I was expecting to see my name listed on
the [contributors page
ZENOTME commented on code in PR #1383:
URL: https://github.com/apache/iceberg-rust/pull/1383#discussion_r2111914945
##
crates/iceberg/src/error.rs:
##
@@ -134,6 +134,8 @@ pub struct Error {
source: Option,
backtrace: Backtrace,
+
+retryable: bool,
Review Comment
ajantha-bhat commented on code in PR #13163:
URL: https://github.com/apache/iceberg/pull/13163#discussion_r2111918206
##
core/src/main/java/org/apache/iceberg/PartitionStatsHandler.java:
##
@@ -336,16 +336,29 @@ private static PartitionMap
computeStatsDiff(
Sets.newHas
ZENOTME closed issue #964: Support commit retrie
URL: https://github.com/apache/iceberg-rust/issues/964
--
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:
JeonDaehong commented on PR #13165:
URL: https://github.com/apache/iceberg/pull/13165#issuecomment-2916382580
> Merged to main. Thanks @JeonDaehong for the backport and @nastra for the
review.
>
> > After my previous PR was merged, I was expecting to see my name listed
on the [contri
c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2112113418
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3964,6 +4172,302 @@ components:
metadata:
$ref: '#/components/schemas/TableMetadata'
+EventsR
c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2112129175
##
open-api/rest-catalog-open-api.py:
##
@@ -1389,6 +1558,101 @@ class CommitTableResponse(BaseModel):
metadata: TableMetadata
+class EventsResponse(BaseMode
rodmeneses commented on code in PR #12071:
URL: https://github.com/apache/iceberg/pull/12071#discussion_r2112131384
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergSink.java:
##
@@ -645,72 +711,135 @@ private DataStream
distributeDataStream(DataStream in
c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2112110143
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3405,6 +3488,131 @@ components:
allOf:
- $ref: '#/components/schemas/ScanTasks'
+Actor:
+ ty
anesterenok commented on issue #12266:
URL: https://github.com/apache/iceberg/issues/12266#issuecomment-2916672959
While I quite in favor of deprecating int96 storage, I must say that its
only a projection of Iceberg types.
If one uses Iceberg `timestamp` type, the value must be considere
rodmeneses commented on code in PR #12071:
URL: https://github.com/apache/iceberg/pull/12071#discussion_r2112131384
##
flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/sink/IcebergSink.java:
##
@@ -645,72 +711,135 @@ private DataStream
distributeDataStream(DataStream in
manuzhang commented on code in PR #12844:
URL: https://github.com/apache/iceberg/pull/12844#discussion_r2112148631
##
api/src/main/java/org/apache/iceberg/actions/RewriteTablePath.java:
##
@@ -86,6 +86,16 @@ public interface RewriteTablePath extends
Action
pvary merged PR #13127:
URL: https://github.com/apache/iceberg/pull/13127
--
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.apa
pvary commented on PR #13127:
URL: https://github.com/apache/iceberg/pull/13127#issuecomment-2916483984
Merged to main.
Thanks @manuzhang for the PR and @mxm for the review!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub a
raulcd commented on code in PR #107:
URL: https://github.com/apache/iceberg-cpp/pull/107#discussion_r2112029921
##
.github/workflows/sanitizer_test.yml:
##
@@ -0,0 +1,63 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.
c-thiel commented on code in PR #12584:
URL: https://github.com/apache/iceberg/pull/12584#discussion_r2112105575
##
open-api/rest-catalog-open-api.yaml:
##
@@ -3041,6 +3096,34 @@ components:
- $ref: '#/components/schemas/AddViewVersionUpdate'
- $ref: '#/compone
RussellSpitzer commented on issue #12751:
URL: https://github.com/apache/iceberg/issues/12751#issuecomment-2916840926
Is this an issue for Datafiles written with Avro Data instead of Parquet?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
RussellSpitzer merged PR #13177:
URL: https://github.com/apache/iceberg/pull/13177
--
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...@ic
angelo-DNAStack commented on PR #12743:
URL: https://github.com/apache/iceberg/pull/12743#issuecomment-2917511977
Checking in on this PR again, please let me know if there are any additional
changes required.
--
This is an automated message from the Apache Git Service.
To respond to the m
RussellSpitzer commented on PR #13173:
URL: https://github.com/apache/iceberg/pull/13173#issuecomment-2917518749
Thanks @ajantha-bhat and @danielcweeks 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
RussellSpitzer merged PR #13173:
URL: https://github.com/apache/iceberg/pull/13173
--
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...@ic
RussellSpitzer commented on PR #13177:
URL: https://github.com/apache/iceberg/pull/13177#issuecomment-2917517509
Thanks @ajantha-bhat , @flyrain + @singhpk234 for reviews!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and us
RussellSpitzer merged PR #13176:
URL: https://github.com/apache/iceberg/pull/13176
--
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...@ic
RussellSpitzer commented on PR #13176:
URL: https://github.com/apache/iceberg/pull/13176#issuecomment-2917516161
Thanks @ajantha-bhat + @danielcweeks 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
RussellSpitzer merged PR #13172:
URL: https://github.com/apache/iceberg/pull/13172
--
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...@ic
RussellSpitzer commented on PR #13172:
URL: https://github.com/apache/iceberg/pull/13172#issuecomment-2917520514
Thanks @ajantha-bhat + @danielcweeks 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
RussellSpitzer commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r2112716340
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -107,9 +115,12 @@ public abstract class TestRemoveOr
RussellSpitzer commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r2112725064
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -344,6 +374,8 @@ public void testWapFilesAreKept() {
RussellSpitzer commented on code in PR #12254:
URL: https://github.com/apache/iceberg/pull/12254#discussion_r2112725064
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRemoveOrphanFilesAction.java:
##
@@ -344,6 +374,8 @@ public void testWapFilesAreKept() {
nastra commented on code in PR #13143:
URL: https://github.com/apache/iceberg/pull/13143#discussion_r2112280095
##
core/src/test/java/org/apache/iceberg/catalog/CatalogTests.java:
##
@@ -959,6 +959,16 @@ public void testLoadMissingTable() {
.hasMessageStartingWith("Tabl
RussellSpitzer opened a new pull request, #13173:
URL: https://github.com/apache/iceberg/pull/13173
(no comment)
--
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 unsubscr
huaxingao merged PR #13168:
URL: https://github.com/apache/iceberg/pull/13168
--
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
huaxingao commented on PR #13168:
URL: https://github.com/apache/iceberg/pull/13168#issuecomment-2916961579
Merged. Thanks @wypoon for the PR! Thanks @singhpk234 for the review!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
RussellSpitzer opened a new pull request, #13172:
URL: https://github.com/apache/iceberg/pull/13172
(no comment)
--
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 unsubscr
vishwanthp12 opened a new issue, #13171:
URL: https://github.com/apache/iceberg/issues/13171
Command I'm using in Spark-sql: CALL
iceberg_catalog.system.refresh_table('iceberg_catalog.$db.$table');
I'm using JDBC catalog configured on MariaDB (MYSQL)
here is my spark-default.co
ajantha-bhat commented on code in PR #13123:
URL: https://github.com/apache/iceberg/pull/13123#discussion_r2112311313
##
api/src/test/java/org/apache/iceberg/TestHelpers.java:
##
@@ -54,7 +54,7 @@ public class TestHelpers {
private TestHelpers() {}
- public static final
ajantha-bhat opened a new pull request, #13175:
URL: https://github.com/apache/iceberg/pull/13175
voted on mailing list:
https://lists.apache.org/thread/9oncq0j0222nrm6scvm89xdq3gozg24p
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on t
1 - 100 of 184 matches
Mail list logo
