[I] Python defined schema does not match created schema [iceberg-python]

2025-02-26 Thread via GitHub
grbinho opened a new issue, #1728: URL: https://github.com/apache/iceberg-python/issues/1728 ### Apache Iceberg version 0.8.1 (latest release) ### Please describe the bug 🐞 Hi, I'm using pyiceberg with Glue and S3 and while creating a table that contains list type

[PR] Spark 3.5: Adjust repeated INFO logs to DEBUG in SparkWrite and SparkPositionDeltaWrite [iceberg]

2025-02-26 Thread via GitHub
manuzhang opened a new pull request, #12404: URL: https://github.com/apache/iceberg/pull/12404 I see repeated logs as follows from `SparkWrite` in Spark driver. ``` 25/02/26 01:48:56 INFO SparkWrite: Requesting 0 bytes advisory partition size for table *** 25/02/26 01:48:56 INFO

Re: [PR] feat: support `arrow_struct_to_iceberg_struct` [iceberg-rust]

2025-02-26 Thread via GitHub
liurenjie1024 merged PR #731: URL: https://github.com/apache/iceberg-rust/pull/731 -- 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

Re: [PR] Parquet: Implement Variant writers [iceberg]

2025-02-26 Thread via GitHub
aihuaxu commented on code in PR #12323: URL: https://github.com/apache/iceberg/pull/12323#discussion_r1972563111 ## core/src/main/java/org/apache/iceberg/variants/VariantVisitor.java: ## @@ -0,0 +1,83 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or m

Re: [PR] feat: support position delete writer [iceberg-rust]

2025-02-26 Thread via GitHub
ZENOTME commented on code in PR #704: URL: https://github.com/apache/iceberg-rust/pull/704#discussion_r1972829889 ## crates/iceberg/src/writer/base_writer/position_delete_file_writer.rs: ## @@ -0,0 +1,277 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or m

Re: [I] Support accessors for ManifestMetadata fields [iceberg-rust]

2025-02-26 Thread via GitHub
liurenjie1024 closed issue #1012: Support accessors for ManifestMetadata fields URL: https://github.com/apache/iceberg-rust/issues/1012 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

Re: [I] Support accessors for ManifestMetadata fields [iceberg-rust]

2025-02-26 Thread via GitHub
liurenjie1024 closed issue #1012: Support accessors for ManifestMetadata fields URL: https://github.com/apache/iceberg-rust/issues/1012 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific co

Re: [PR] chore(spec): add accessor methods for ManifestMetadata [iceberg-rust]

2025-02-26 Thread via GitHub
liurenjie1024 merged PR #1013: URL: https://github.com/apache/iceberg-rust/pull/1013 -- 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...@

Re: [PR] Spark: Detect dangling DVs properly [iceberg]

2025-02-26 Thread via GitHub
singhpk234 commented on code in PR #12270: URL: https://github.com/apache/iceberg/pull/12270#discussion_r1972008956 ## spark/v3.5/spark/src/main/java/org/apache/iceberg/spark/actions/RemoveDanglingDeletesSparkAction.java: ## @@ -156,7 +162,12 @@ private List findDanglingDeletes(

Re: [PR] Data: Add partition stats writer and reader [iceberg]

2025-02-26 Thread via GitHub
ajantha-bhat commented on PR #11216: URL: https://github.com/apache/iceberg/pull/11216#issuecomment-2687054193 @RussellSpitzer, @aokolnychyi: Anymore comments? others have approved the changes. -- This is an automated message from the Apache Git Service. To respond to the message, plea

Re: [PR] Core,REST: extend httpClient builder to support tls factory [iceberg]

2025-02-26 Thread via GitHub
munendrasn commented on code in PR #11979: URL: https://github.com/apache/iceberg/pull/11979#discussion_r1973048206 ## core/src/main/java/org/apache/iceberg/rest/HTTPClient.java: ## @@ -480,6 +486,13 @@ public Builder withObjectMapper(ObjectMapper objectMapper) { return

Re: [PR] Core,REST: extend httpClient builder to support tls factory [iceberg]

2025-02-26 Thread via GitHub
munendrasn commented on code in PR #11979: URL: https://github.com/apache/iceberg/pull/11979#discussion_r1973048206 ## core/src/main/java/org/apache/iceberg/rest/HTTPClient.java: ## @@ -480,6 +486,13 @@ public Builder withObjectMapper(ObjectMapper objectMapper) { return

Re: [PR] feat: support delete if empty for parquet writer [iceberg-rust]

2025-02-26 Thread via GitHub
ZENOTME commented on PR #838: URL: https://github.com/apache/iceberg-rust/pull/838#issuecomment-2686827200 I think this PR is ready to review. cc @Xuanwo @liurenjie1024 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] support create table like in flink catalog [iceberg]

2025-02-26 Thread via GitHub
swapna267 commented on code in PR #12199: URL: https://github.com/apache/iceberg/pull/12199#discussion_r1972866279 ## flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/FlinkCreateTableOptions.java: ## @@ -0,0 +1,114 @@ +/* + * Licensed to the Apache Software Foundation (A

Re: [PR] support create table like in flink catalog [iceberg]

2025-02-26 Thread via GitHub
swapna267 commented on code in PR #12199: URL: https://github.com/apache/iceberg/pull/12199#discussion_r1972877314 ## flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/TestFlinkCatalogTable.java: ## @@ -188,6 +188,37 @@ public void testCreateTableLike() throws TableNotEx

Re: [PR] support create table like in flink catalog [iceberg]

2025-02-26 Thread via GitHub
swapna267 commented on code in PR #12199: URL: https://github.com/apache/iceberg/pull/12199#discussion_r1972880531 ## flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/TestFlinkCatalogTable.java: ## @@ -188,6 +188,37 @@ public void testCreateTableLike() throws TableNotEx

Re: [PR] support create table like in flink catalog [iceberg]

2025-02-26 Thread via GitHub
swapna267 commented on code in PR #12199: URL: https://github.com/apache/iceberg/pull/12199#discussion_r1972885322 ## flink/v1.20/flink/src/test/java/org/apache/iceberg/flink/source/TestIcebergSourceSql.java: ## @@ -162,4 +162,21 @@ public void testWatermarkOptionsDescending() t

Re: [PR] support create table like in flink catalog [iceberg]

2025-02-26 Thread via GitHub
swapna267 commented on code in PR #12199: URL: https://github.com/apache/iceberg/pull/12199#discussion_r1972887391 ## flink/v1.20/flink/src/main/java/org/apache/iceberg/flink/FlinkCatalog.java: ## @@ -332,7 +335,22 @@ public List listTables(String databaseName) public Catalog

Re: [PR] Auth Manager API part 6: API enablement [iceberg]

2025-02-26 Thread via GitHub
ajantha-bhat commented on PR #12197: URL: https://github.com/apache/iceberg/pull/12197#issuecomment-2687048478 @danielcweeks and @nastra: Can we please start the review on this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

<    1   2