manuzhang commented on PR #11643:
URL: https://github.com/apache/iceberg/pull/11643#issuecomment-2538063691
@nastra any more comments?
--
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
nastra merged PR #11729:
URL: https://github.com/apache/iceberg/pull/11729
--
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 #11759:
URL: https://github.com/apache/iceberg/pull/11759
--
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 commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881528468
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/AzureSasCredentialRefresher.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
nastra commented on PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#issuecomment-2538047119
@danielcweeks could you also take a look at this PR please?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
U
Fokko commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881537732
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated to wi
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881537532
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/VendedAdlsCredentialProvider.java:
##
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881531755
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/AzureSasCredentialRefresher.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881533944
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/VendedAdlsCredentialProvider.java:
##
@@ -0,0 +1,197 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881531528
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/AzureSasCredentialRefresher.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881531305
##
azure/src/main/java/org/apache/iceberg/azure/adlsv2/AzureSasCredentialRefresher.java:
##
@@ -0,0 +1,71 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) u
nastra commented on code in PR #11577:
URL: https://github.com/apache/iceberg/pull/11577#discussion_r1881527842
##
azure/src/main/java/org/apache/iceberg/azure/AzureProperties.java:
##
@@ -90,7 +117,9 @@ public Optional adlsWriteBlockSize() {
*/
public void applyClientCon
ZENOTME commented on PR #479:
URL: https://github.com/apache/iceberg-rust/pull/479#issuecomment-2538033235
Seems this PR affects the result type of Year, Month, Day transform. Should
we only change the result type of Transform::Day.🤔
--
This is an automated message from the Apache Git Ser
nastra commented on code in PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#discussion_r1881520238
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -149,6 +149,7 @@ public class RESTSessionCatalog extends
BaseViewSessionCatalog
nastra commented on code in PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#discussion_r1881520238
##
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##
@@ -149,6 +149,7 @@ public class RESTSessionCatalog extends
BaseViewSessionCatalog
nastra commented on code in PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#discussion_r1881519406
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -70,6 +85,7 @@ public class Endpoint {
public static final Endpoint V1_DELETE_VIEW = Endpoint.cre
xxchan commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881514203
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated to w
ZENOTME commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881503123
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated to
nikulaja commented on issue #11733:
URL: https://github.com/apache/iceberg/issues/11733#issuecomment-2538002097
@Fokko Thank you for reply! Seems very reasonable. Could this information be
added to documentation? I'm very new to iceberg so wouldn't know if this is a
regular thing with other
ZENOTME commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881503123
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated to
wypoon commented on PR #11661:
URL: https://github.com/apache/iceberg/pull/11661#issuecomment-2537946114
Hi @nastra, I have addressed your feedback last week. Can you please review
this again?
--
This is an automated message from the Apache Git Service.
To respond to the message, please l
wenym1 commented on PR #760:
URL: https://github.com/apache/iceberg-rust/pull/760#issuecomment-2537946595
> I find that the implementation has some problems now, it will cause
recursive calls endlessly and stack overflow finally.
>
> Reproduce:
>
> ```
>#[tokio::test]
pvary commented on PR #11750:
URL: https://github.com/apache/iceberg/pull/11750#issuecomment-2537927187
Will check next week if it is not merged till then
--
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
Ppei-Wang commented on code in PR #11728:
URL: https://github.com/apache/iceberg/pull/11728#discussion_r1881446544
##
api/src/main/java/org/apache/iceberg/types/Types.java:
##
@@ -412,6 +413,24 @@ public String toString() {
}
}
+ public static class NullType extends P
ZENOTME commented on PR #760:
URL: https://github.com/apache/iceberg-rust/pull/760#issuecomment-2537842348
I find that the implementation has some problems now, it will cause
recursive calls endlessly and stack overflow finally.
Reproduce:
```
#[tokio::test]
async fn
xxchan commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881355866
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated to w
pawansanz commented on issue #1418:
URL:
https://github.com/apache/iceberg-python/issues/1418#issuecomment-2537769371
can anyone help with original issue that was reported regarding "Apache
Iceberg version"
--
This is an automated message from the Apache Git Service.
To respond to the me
wzx140 commented on code in PR #11748:
URL: https://github.com/apache/iceberg/pull/11748#discussion_r1881295827
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewritePositionDeleteFilesAction.java:
##
@@ -275,7 +275,7 @@ public void testRewriteFilter() th
manuzhang closed pull request #11654: Core, Spark3.5: Fix tests failure due to
timeout
URL: https://github.com/apache/iceberg/pull/11654
--
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
Samreay commented on PR #7914:
URL: https://github.com/apache/iceberg/pull/7914#issuecomment-2537695273
Has anyone got a nice workaround for how to remove orphan files for an
S3-located iceberg table?
--
This is an automated message from the Apache Git Service.
To respond to the message,
ebyhr opened a new pull request, #11758:
URL: https://github.com/apache/iceberg/pull/11758
I propose adding query runners in tests so developers can debug Spark SQL
without local jar publish.
This is similar to query runners in Trino project. The project provides
several query runne
ConeyLiu opened a new pull request, #1427:
URL: https://github.com/apache/iceberg-python/pull/1427
Now, we only support plan files. Plan tasks(split large file based on
split_offset) would be more useful when we want to read in parallel.
--
This is an automated message from the Apache Git
liurenjie1024 commented on PR #761:
URL: https://github.com/apache/iceberg-rust/pull/761#issuecomment-2537679283
Hi, @wenym1 I saw you submitted several similar refactoring to current api.
While the community appreciate your contribution, could you open an issue to
raise discussion about th
wzx140 commented on code in PR #11748:
URL: https://github.com/apache/iceberg/pull/11748#discussion_r1881295827
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewritePositionDeleteFilesAction.java:
##
@@ -275,7 +275,7 @@ public void testRewriteFilter() th
liurenjie1024 opened a new pull request, #779:
URL: https://github.com/apache/iceberg-rust/pull/779
Add a protected branch rule to dismiss stale reviews when new commits are
added.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to Git
mkaravel commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1881291822
##
format/spec.md:
##
@@ -603,6 +608,10 @@ Notes:
4. Position delete metadata can use `referenced_data_file` when all deletes
tracked by the entry are in a single
mkaravel commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1881291822
##
format/spec.md:
##
@@ -603,6 +608,10 @@ Notes:
4. Position delete metadata can use `referenced_data_file` when all deletes
tracked by the entry are in a single
zhjwpku commented on code in PR #5:
URL: https://github.com/apache/iceberg-cpp/pull/5#discussion_r1881282329
##
cmake-format.py:
##
@@ -0,0 +1,74 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# d
zhjwpku commented on code in PR #4:
URL: https://github.com/apache/iceberg-cpp/pull/4#discussion_r1881257096
##
.clang-format:
##
@@ -0,0 +1,22 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements. See the NOTICE file
+# dis
ebyhr commented on code in PR #11755:
URL: https://github.com/apache/iceberg/pull/11755#discussion_r1881237568
##
core/src/test/java/org/apache/iceberg/TestSchemaUpdate.java:
##
@@ -731,6 +731,17 @@ public void testAmbiguousAdd() {
.hasMessageStartingWith("Cannot add co
mkaravel commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1879219879
##
format/spec.md:
##
@@ -205,13 +205,18 @@ Supported primitive types are defined in the table below.
Primitive types added
| | **`uuid`**
manuzhang commented on PR #11737:
URL: https://github.com/apache/iceberg/pull/11737#issuecomment-2537563264
@ajantha-bhat @RussellSpitzer @nastra Please help review whether this is a
valid request.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
601madman opened a new pull request, #11757:
URL: https://github.com/apache/iceberg/pull/11757
### Problem
When IDENTIFIER FIELDS are set, and `merge-on-read` mode is used, a
validation error occurs due to incorrect metadata schema checks.
### Solution
Adjusted the `calculateMet
manuzhang commented on code in PR #11749:
URL: https://github.com/apache/iceberg/pull/11749#discussion_r1881244151
##
format/spec.md:
##
@@ -454,7 +454,7 @@ Partition field IDs must be reused if an existing partition
spec contains an equ
| **`truncate[W]`** | Value truncated t
manuzhang commented on PR #10996:
URL: https://github.com/apache/iceberg/pull/10996#issuecomment-2537550479
We have a [consensus on the dev
list](https://lists.apache.org/thread/jfcqfw9vhq4j7h0kwnlf338jgyzcq8s4) to drop
hive-runtime and upgrade to Hive 4. I've submitted
https://github.com/
ebyhr commented on code in PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#discussion_r1881226290
##
core/src/main/java/org/apache/iceberg/rest/Endpoint.java:
##
@@ -61,6 +63,19 @@ public class Endpoint {
Endpoint.create("POST", ResourcePaths.V1_TABLE_REGISTE
github-actions[bot] commented on issue #175:
URL: https://github.com/apache/iceberg-python/issues/175#issuecomment-2537469241
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 issue #601:
URL: https://github.com/apache/iceberg-python/issues/601#issuecomment-2537469223
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 Apac
github-actions[bot] closed issue #601: Create iceberg table from existsing
parquet files with slightly different schemas (schemas merge is possible).
URL: https://github.com/apache/iceberg-python/issues/601
--
This is an automated message from the Apache Git Service.
To respond to the message
github-actions[bot] closed pull request #11196: update PartitionSpec with
snapshot'schema
URL: https://github.com/apache/iceberg/pull/11196
--
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
github-actions[bot] closed pull request #11280: Core: Store schema and spec in
TaskContext to avoid unnecessary deserialization (#11235)
URL: https://github.com/apache/iceberg/pull/11280
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
github-actions[bot] commented on issue #10491:
URL: https://github.com/apache/iceberg/issues/10491#issuecomment-2537466332
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 #9736:
URL: https://github.com/apache/iceberg/issues/9736#issuecomment-2537466213
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] commented on PR #11288:
URL: https://github.com/apache/iceberg/pull/11288#issuecomment-2537466505
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 #11196:
URL: https://github.com/apache/iceberg/pull/11196#issuecomment-2537466436
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
github-actions[bot] commented on PR #11280:
URL: https://github.com/apache/iceberg/pull/11280#issuecomment-2537466476
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
github-actions[bot] commented on issue #10497:
URL: https://github.com/apache/iceberg/issues/10497#issuecomment-2537466356
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] closed issue #9736: Merge into using the exactly dataset
copy the entire data
URL: https://github.com/apache/iceberg/issues/9736
--
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 t
rdblue commented on code in PR #11415:
URL: https://github.com/apache/iceberg/pull/11415#discussion_r1881180413
##
core/src/main/java/org/apache/iceberg/variants/PrimitiveWrapper.java:
##
@@ -0,0 +1,206 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or
rdblue commented on code in PR #11317:
URL: https://github.com/apache/iceberg/pull/11317#discussion_r1881174754
##
spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/SparkCatalog.java:
##
@@ -365,6 +370,14 @@ public boolean purgeTable(Identifier ident) {
String metad
rdblue commented on code in PR #11317:
URL: https://github.com/apache/iceberg/pull/11317#discussion_r1881170392
##
core/src/main/java/org/apache/iceberg/CatalogProperties.java:
##
@@ -78,6 +78,14 @@ private CatalogProperties() {}
public static final boolean IO_MANIFEST_CACH
jqin61 commented on PR #931:
URL: https://github.com/apache/iceberg-python/pull/931#issuecomment-2537415525
Thanks for fixing the jar issue, shall we rerun CI and merge? @Fokko Thank
you!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881106923
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881106567
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881105847
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881104193
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881103731
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881105376
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881105008
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11660:
URL: https://github.com/apache/iceberg/pull/11660#discussion_r1881103356
##
format/spec.md:
##
@@ -693,6 +686,64 @@ A snapshot's `first-row-id` is assigned to the table's
current `next-row-id` on
The snapshot's `first-row-id` is t
RussellSpitzer commented on code in PR #11365:
URL: https://github.com/apache/iceberg/pull/11365#discussion_r1881100218
##
format/view-spec.md:
##
@@ -97,7 +97,10 @@ Summary is a string to string map of metadata about a view
version. Common metad
View definitions can be repr
wmoustafa commented on code in PR #11365:
URL: https://github.com/apache/iceberg/pull/11365#discussion_r1881085717
##
format/view-spec.md:
##
@@ -97,7 +97,10 @@ Summary is a string to string map of metadata about a view
version. Common metad
View definitions can be represent
ajreid21 commented on PR #11756:
URL: https://github.com/apache/iceberg/pull/11756#issuecomment-2537316670
@nastra
--
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 unsub
jimmyxie-figma commented on issue #357:
URL: https://github.com/apache/iceberg-python/issues/357#issuecomment-2537304523
Any update on supporting distributed write, we are also interested in adding
iceberg write capability to Ray.
https://github.com/ray-project/ray/issues/49032
--
Thi
zeroshade commented on PR #176:
URL: https://github.com/apache/iceberg-go/pull/176#issuecomment-2537284504
I should be able to give this a review tomorrow or Friday. In the meantime
can you resolve the conflict in the go.mod? Thanks!
--
This is an automated message from the Apache Git Ser
kazuyukitanimura commented on issue #11741:
URL: https://github.com/apache/iceberg/issues/11741#issuecomment-2537281522
@sunny1154 I think you would need to specify the catalog in
`TableIdentifier()`
Otherwise, Spark tries to use `spark_catalog`
https://github.com/apache/spark/blo
kazuyukitanimura commented on issue #11741:
URL: https://github.com/apache/iceberg/issues/11741#issuecomment-2537255407
Just to add @huaxingao's point
tableExists(dbName: String, tableName: String): Boolean
it is meant to be only for the hardcoded spark_catalog only. But looks like
sunny1154 commented on issue #11741:
URL: https://github.com/apache/iceberg/issues/11741#issuecomment-2537250201
thanks @huaxingao for looking into this.
is `spark.sessionState.catalog.getTableMetadata(TableIdentifier(table,
Some(database)))` also expected to work with HMS? currently
sungwy commented on code in PR #1422:
URL: https://github.com/apache/iceberg-python/pull/1422#discussion_r1881033971
##
tests/test_transforms.py:
##
@@ -988,608 +997,367 @@ def _test_projection(lhs:
Optional[UnboundPredicate[L]], rhs: Optional[UnboundPr
raise ValueErro
RussellSpitzer commented on code in PR #11365:
URL: https://github.com/apache/iceberg/pull/11365#discussion_r1881029096
##
format/view-spec.md:
##
@@ -97,7 +97,10 @@ Summary is a string to string map of metadata about a view
version. Common metad
View definitions can be repr
RussellSpitzer commented on code in PR #11365:
URL: https://github.com/apache/iceberg/pull/11365#discussion_r1881029096
##
format/view-spec.md:
##
@@ -97,7 +97,10 @@ Summary is a string to string map of metadata about a view
version. Common metad
View definitions can be repr
Fokko commented on PR #1425:
URL: https://github.com/apache/iceberg-python/pull/1425#issuecomment-2537217676
Thanks @sungwy 🙌
--
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
Fokko merged PR #1425:
URL: https://github.com/apache/iceberg-python/pull/1425
--
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...@iceber
loicalleyne commented on PR #176:
URL: https://github.com/apache/iceberg-go/pull/176#issuecomment-2537210180
@zeroshade hoping you can review when you've got time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
UR
amogh-jahagirdar merged PR #11753:
URL: https://github.com/apache/iceberg/pull/11753
--
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...@
Fokko commented on PR #1362:
URL: https://github.com/apache/iceberg-python/pull/1362#issuecomment-2537057846
@hgollakota It looks like there is a formatting issue, could you run `make
lint`? :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please
Fokko commented on PR #1393:
URL: https://github.com/apache/iceberg-python/pull/1393#issuecomment-2537019109
This is actually dangerous in the case of V1 tables.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
Fokko closed pull request #1393: Ignore partition fields that reference a
dropped source-id
URL: https://github.com/apache/iceberg-python/pull/1393
--
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
singhpk234 opened a new pull request, #11755:
URL: https://github.com/apache/iceberg/pull/11755
## About the change
Recently stumbled on a schema where a column was of struct type but the
underlying struct was empty, this lead to failure when writing the parquet file
because :
sungwy commented on PR #1423:
URL: https://github.com/apache/iceberg-python/pull/1423#issuecomment-2536972352
Yes, I think updating this PR to include the changes for both makes sense
@jiakai-li 👍
Thank you again for tackling this issue!
--
This is an automated message from the Ap
szehon-ho commented on code in PR #11748:
URL: https://github.com/apache/iceberg/pull/11748#discussion_r1880821876
##
spark/v3.5/spark/src/test/java/org/apache/iceberg/spark/actions/TestRewritePositionDeleteFilesAction.java:
##
@@ -275,7 +275,7 @@ public void testRewriteFilter()
soumya-ghosh commented on PR #1241:
URL: https://github.com/apache/iceberg-python/pull/1241#issuecomment-2536965547
@Fokko bumping this up 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
ryzhyk commented on issue #405:
URL: https://github.com/apache/iceberg-rust/issues/405#issuecomment-2536958987
Thanks for the update @sdd !
--
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
wmoustafa commented on code in PR #11365:
URL: https://github.com/apache/iceberg/pull/11365#discussion_r1880814678
##
format/view-spec.md:
##
@@ -97,7 +97,10 @@ Summary is a string to string map of metadata about a view
version. Common metad
View definitions can be represent
sdd commented on issue #405:
URL: https://github.com/apache/iceberg-rust/issues/405#issuecomment-2536941270
Hi @ryzhyk - I implemented the default value handling and type promotion
limitations mentioned in @liurenjie1024's [comment at the top of the
issue](https://github.com/apache/iceberg-
jiakai-li commented on PR #1423:
URL: https://github.com/apache/iceberg-python/pull/1423#issuecomment-2536887038
Thanks very much for the guidance guys @sungwy and @Fokko . Is it ok for me
to pick up the delete part as well? I'll update this PR to include both
operations if that's ok. Thank
dramaticlly commented on issue #11741:
URL: https://github.com/apache/iceberg/issues/11741#issuecomment-2536868437
> After taking a closer look at the [Java
Doc](https://github.com/apache/spark/blob/branch-3.5/sql/core/src/main/scala/org/apache/spark/sql/catalog/Catalog.scala#L224),
I found
Sl1mb0 opened a new issue, #778:
URL: https://github.com/apache/iceberg-rust/issues/778
At the moment, the building and serialization of Iceberg metadata is coupled
together.
For example, let's say I want to build a `ManifestFile` that I then add to a
`ManifestList`:
(some cod
RussellSpitzer commented on PR #11317:
URL: https://github.com/apache/iceberg/pull/11317#issuecomment-2536685955
https://docs.google.com/document/d/1iPGVCIcr-M0XtAiudOguWAvmqIdVgpYN5vz5ohO8PKw/edit?tab=t.0#heading=h.cr6o1g2rn5hc
--
This is an automated message from the Apache Git Service.
c-thiel opened a new issue, #11754:
URL: https://github.com/apache/iceberg/issues/11754
### Feature Request / Improvement
Currently when purge-dropping tables with Spark and the REST Catalog, Spark
deletes all files of the tables before sending the drop request to the REST
Catalog. I
1 - 100 of 165 matches
Mail list logo
