Re: [PR] Spark: Remove closing of IO in SerializableTable* [iceberg]

2025-02-02 Thread via GitHub
mgmarino commented on PR #12129: URL: https://github.com/apache/iceberg/pull/12129#issuecomment-2630199667 @aokolnychyi Any chance you'll be able to give some feedback on this? Thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] chore: use shared containers for integration tests [iceberg-rust]

2025-02-02 Thread via GitHub
gruuya commented on PR #924: URL: https://github.com/apache/iceberg-rust/pull/924#issuecomment-2630198784 > thanks! generally lgtm. Does this also affect CI runs? if so, would be great to also include how much time we saved there too. Good question; a brief glance suggests it may have

Re: [PR] spec: Remove `source-ids` for `V{1,2}` tables [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on PR #12161: URL: https://github.com/apache/iceberg/pull/12161#issuecomment-2630172217 By the way I gonna start impl side of the thing :) -- 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

Re: [I] Namespace names with dot(.) not supported by JDBC catalog when listing namespaces [iceberg]

2025-02-02 Thread via GitHub
mingnuj commented on issue #11990: URL: https://github.com/apache/iceberg/issues/11990#issuecomment-2630158964 I gave it some more thought, and I was wondering if you could explain why the function that splits based on dots is located in `JdbcUtil.java`. In my opinion, since namespace

Re: [PR] spec: Remove `source-ids` for `V{1,2}` tables [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12161: URL: https://github.com/apache/iceberg/pull/12161#discussion_r1938866471 ## format/spec.md: ## @@ -1400,7 +1400,7 @@ In some cases partition specs are stored using only the field list instead of th The `field-id` property was added fo

Re: [PR] spec: Remove `source-ids` for `V{1,2}` tables [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on PR #12161: URL: https://github.com/apache/iceberg/pull/12161#issuecomment-2630102760 @Fokko Thanks ! As discussed together, I was about to propose this, so totally agree ! -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] [infra] refactor release github workflow [iceberg-python]

2025-02-02 Thread via GitHub
Fokko merged PR #1600: URL: https://github.com/apache/iceberg-python/pull/1600 -- 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

Re: [PR] [infra] speed up artifact build with uv [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1599: URL: https://github.com/apache/iceberg-python/pull/1599#discussion_r1938854065 ## .github/workflows/python-release.yml: ## @@ -147,23 +147,23 @@ jobs: run: python3 -m poetry build --format=sdist if: startsWith(matrix.os, 'ub

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938853753 ## gcp-bundle/LICENSE: ## @@ -220,100 +220,90 @@ License: Apache 2.0 - http://www.apache.org/licenses/LICENSE-2.0 --

[PR] spec: Remove `source-ids` for `V{1,2}` tables [iceberg]

2025-02-02 Thread via GitHub
Fokko opened a new pull request, #12161: URL: https://github.com/apache/iceberg/pull/12161 I was looking at adding support for `source-ids` in PyIceberg, but noticed that it was also still lacking for Java. I've noticed that `source-ids` are also backported to V1 and V2 tables, which

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-02 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1938207738 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [PR] Docs: Add rewrite-table-path in spark procedure [iceberg]

2025-02-02 Thread via GitHub
szehon-ho commented on code in PR #12115: URL: https://github.com/apache/iceberg/pull/12115#discussion_r1938841961 ## docs/docs/spark-procedures.md: ## @@ -972,4 +972,91 @@ CALL catalog_name.system.compute_table_stats(table => 'my_table', snapshot_id => Collect statistics of t

Re: [I] Namespace names with dot(.) not supported by JDBC catalog when listing namespaces [iceberg]

2025-02-02 Thread via GitHub
mingnuj commented on issue #11990: URL: https://github.com/apache/iceberg/issues/11990#issuecomment-2630077016 I am attaching the solution I personally implemented based on the provided guidance, as someone else might encounter the same issue. I wanted the namespace to allow dots not

Re: [I] Namespace names with dot(.) not supported by JDBC catalog when listing namespaces [iceberg]

2025-02-02 Thread via GitHub
mingnuj closed issue #11990: Namespace names with dot(.) not supported by JDBC catalog when listing namespaces URL: https://github.com/apache/iceberg/issues/11990 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

Re: [PR] [infra] speed up artifact build with uv [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1599: URL: https://github.com/apache/iceberg-python/pull/1599#discussion_r1938821618 ## .github/workflows/python-release.yml: ## @@ -147,23 +147,23 @@ jobs: run: python3 -m poetry build --format=sdist if: startsWith(matrix.os, 'ub

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#issuecomment-2630045458 Any time, sorry for taking so long :3 -- 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

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938786136 ## gcp-bundle/LICENSE: ## @@ -220,100 +220,90 @@ License: Apache 2.0 - http://www.apache.org/licenses/LICENSE-2.0 --

Re: [I] Variant Data Type Support [iceberg]

2025-02-02 Thread via GitHub
jeremytee97 commented on issue #10392: URL: https://github.com/apache/iceberg/issues/10392#issuecomment-2629746790 +1 -- 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 uns

Re: [PR] Core: Delegate `resetMainBranch` to `removeRef` in `TableMetadata::Builder` [iceberg]

2025-02-02 Thread via GitHub
smaheshwar-pltr commented on code in PR #12149: URL: https://github.com/apache/iceberg/pull/12149#discussion_r1938644937 ## core/src/main/java/org/apache/iceberg/TableMetadata.java: ## @@ -1310,13 +1310,7 @@ public Builder removeRef(String name) { } private Builder r

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
amogh-jahagirdar commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938632389 ## gcp-bundle/LICENSE: ## @@ -220,100 +220,90 @@ License: Apache 2.0 - http://www.apache.org/licenses/LICENSE-2.0 --

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
amogh-jahagirdar commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938632389 ## gcp-bundle/LICENSE: ## @@ -220,100 +220,90 @@ License: Apache 2.0 - http://www.apache.org/licenses/LICENSE-2.0 --

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
amogh-jahagirdar commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938632389 ## gcp-bundle/LICENSE: ## @@ -220,100 +220,90 @@ License: Apache 2.0 - http://www.apache.org/licenses/LICENSE-2.0 --

[PR] [infra] add testpypi nightly build [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu opened a new pull request, #1601: URL: https://github.com/apache/iceberg-python/pull/1601 Had to setup [Publishing to PyPI with a Trusted Publisher](https://docs.pypi.org/trusted-publishers/) on https://test.pypi.org/ Using example of publishing [pyiceberg_core](https://gi

Re: [I] Cannot specify "file://" uri or direct local file location for warehouse in Windows [iceberg-python]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #1005: URL: https://github.com/apache/iceberg-python/issues/1005#issuecomment-2629643989 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

Re: [I] Add an Integration Test Suite for the Rest Catalog interactions [iceberg-python]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #1010: URL: https://github.com/apache/iceberg-python/issues/1010#issuecomment-2629643968 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

Re: [I] support pyarrow recordbatch as a valid data source for writing Iceberg table [iceberg-python]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #1004: URL: https://github.com/apache/iceberg-python/issues/1004#issuecomment-2629644003 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

Re: [PR] Core: Delegate `resetMainBranch` to `removeRef` in `TableMetadata::Builder` [iceberg]

2025-02-02 Thread via GitHub
ebyhr commented on code in PR #12149: URL: https://github.com/apache/iceberg/pull/12149#discussion_r1938612158 ## core/src/main/java/org/apache/iceberg/TableMetadata.java: ## @@ -1310,13 +1310,7 @@ public Builder removeRef(String name) { } private Builder resetMainBr

Re: [PR] Doc:Hive 4.0 and later versions allow vectorized read and write opera… [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on PR #11877: URL: https://github.com/apache/iceberg/pull/11877#issuecomment-2629642300 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

Re: [I] Detecting duplicates in the Flink Data Stream API [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #10683: URL: https://github.com/apache/iceberg/issues/10683#issuecomment-2629642189 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

Re: [PR] Introduce `MissingRequiredFilesToDeleteException` for Streaming Deletes [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on PR #11887: URL: https://github.com/apache/iceberg/pull/11887#issuecomment-2629642330 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

Re: [PR] Spark 3.5: Refactor scanning changelog table with timestamps [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on PR #11612: URL: https://github.com/apache/iceberg/pull/11612#issuecomment-2629642276 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

Re: [I] Field comments are not written for timestamp field [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #4212: URL: https://github.com/apache/iceberg/issues/4212#issuecomment-2629642141 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.

Re: [I] Detecting duplicates in the Flink Data Stream API [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] closed issue #10683: Detecting duplicates in the Flink Data Stream API URL: https://github.com/apache/iceberg/issues/10683 -- 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

Re: [I] Define behavior of gc.enabled and location ownership [iceberg]

2025-02-02 Thread via GitHub
github-actions[bot] commented on issue #4159: URL: https://github.com/apache/iceberg/issues/4159#issuecomment-2629642118 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.

Re: [I] Spark: Add read/write support for UUIDs from bytes [iceberg]

2025-02-02 Thread via GitHub
anuragmantri commented on issue #10635: URL: https://github.com/apache/iceberg/issues/10635#issuecomment-2629626867 Thanks for the ping, I missed this one. I will continue working on this. I tried to add support uuid bytes earlier. Where I was stuck was that the uuid.uuid4().bytes is

Re: [PR] Core: Delegate `resetMainBranch` to `removeRef` in `TableMetadata::Builder` [iceberg]

2025-02-02 Thread via GitHub
ebyhr commented on code in PR #12149: URL: https://github.com/apache/iceberg/pull/12149#discussion_r1938612158 ## core/src/main/java/org/apache/iceberg/TableMetadata.java: ## @@ -1310,13 +1310,7 @@ public Builder removeRef(String name) { } private Builder resetMainBr

Re: [I] Add files to add existing Parquet files to a table [iceberg-rust]

2025-02-02 Thread via GitHub
jonathanc-n commented on issue #932: URL: https://github.com/apache/iceberg-rust/issues/932#issuecomment-2629588992 I would like to try working on 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

[PR] feat: `SnapshotSummary` outline [iceberg-rust]

2025-02-02 Thread via GitHub
jonathanc-n opened a new pull request, #939: URL: https://github.com/apache/iceberg-rust/pull/939 Part of #724. Built the outline of the `SnapshotSummary`. Functionality into the code can be added in a separate pr. -- This is an automated message from the Apache Git Service. To res

[PR] [infra] refactor release github workflow [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu opened a new pull request, #1600: URL: https://github.com/apache/iceberg-python/pull/1600 Moved `svn-build-artifacts` and `svn-merge-artifacts` into `.github/workflows/svn-build-artifacts.yml` Moved `pypi-build-artifacts` and `pypi-merge-artifacts` into `.github/workflows/pypi

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938573122 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,118 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

[PR] [infra] speed up artifact build with uv [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu opened a new pull request, #1599: URL: https://github.com/apache/iceberg-python/pull/1599 `main` branch took 11m10s ([run](https://github.com/kevinjqliu/iceberg-python/actions/runs/13102495444)) This PR branch took 6m13s ([run](https://github.com/kevinjqliu/iceberg-python/acti

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu merged PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391 -- 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...@i

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu commented on PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#issuecomment-2629550063 Thanks for the review @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

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu commented on PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#issuecomment-2629549886 had to add an unrelated change in `mkdocs/docs/api.md`. The url referenced is dead, causing CI to fail -- This is an automated message from the Apache Git Service. To respond

Re: [PR] Adding new rewrite manifest spark action to accept custom partition o… [iceberg]

2025-02-02 Thread via GitHub
zachdisc commented on PR #11881: URL: https://github.com/apache/iceberg/pull/11881#issuecomment-2629545253 @jackye1995 mind taking a look? -- 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 spec

Re: [PR] Spark: Adding simple custom partition sort order option to RewriteManifests Spark Action [iceberg]

2025-02-02 Thread via GitHub
zachdisc closed pull request #9731: Spark: Adding simple custom partition sort order option to RewriteManifests Spark Action URL: https://github.com/apache/iceberg/pull/9731 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Spark: Adding simple custom partition sort order option to RewriteManifests Spark Action [iceberg]

2025-02-02 Thread via GitHub
zachdisc commented on PR #9731: URL: https://github.com/apache/iceberg/pull/9731#issuecomment-2629545161 Closing in favor of https://github.com/apache/iceberg/pull/11881 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938574146 ## mkdocs/docs/how-to-release.md: ## @@ -109,66 +109,81 @@ Create a signed tag: Replace `VERSION` and `RC` with the appropriate values for the release. ```bash

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938573122 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,118 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938572601 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

Re: [I] Support pushdown filters for non-cast date conversion functions (e.g. to_date) [iceberg-rust]

2025-02-02 Thread via GitHub
Fokko commented on issue #933: URL: https://github.com/apache/iceberg-rust/issues/933#issuecomment-2629541746 @omerhadari That would be great, feel free to ping me for a review of reach out in case of any questions 👍 -- This is an automated message from the Apache Git Service. To respond

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938565110 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate"

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938565110 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate"

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
kevinjqliu commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938564633 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate"

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938561415 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938560927 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

Re: [PR] automate building Release Candidate in github action [iceberg-python]

2025-02-02 Thread via GitHub
Fokko commented on code in PR #1391: URL: https://github.com/apache/iceberg-python/pull/1391#discussion_r1938560759 ## .github/workflows/python-release.yml: ## @@ -17,29 +17,192 @@ # under the License. # -name: "Python Release" +name: "Python Build Release Candidate" on:

Re: [PR] Flink: Add null check to writers to prevent resurrecting null values [iceberg]

2025-02-02 Thread via GitHub
mxm commented on code in PR #12049: URL: https://github.com/apache/iceberg/pull/12049#discussion_r1938560173 ## flink/v1.18/flink/src/main/java/org/apache/iceberg/flink/FlinkRowData.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + *

Re: [PR] Build: Bump com.azure:azure-sdk-bom from 1.2.30 to 1.2.31 [iceberg]

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

Re: [PR] chore: make integration tests run concurrently [iceberg-rust]

2025-02-02 Thread via GitHub
kevinjqliu commented on PR #924: URL: https://github.com/apache/iceberg-rust/pull/924#issuecomment-2629508406 thanks! generally lgtm. Does this also affect CI runs? if so, would be great to also include how much time we saved there too. -- This is an automated message from the Apache

Re: [I] Support pushdown filters for non-cast date conversion functions (e.g. to_date) [iceberg-rust]

2025-02-02 Thread via GitHub
omerhadari commented on issue #933: URL: https://github.com/apache/iceberg-rust/issues/933#issuecomment-2629493894 Yes I would like to take a shot at this. Will be free a bit later this week to try if that's ok? -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938543535 ## gcp-bundle/LICENSE: ## @@ -325,24 +315,24 @@ License: The Apache Software License, Version 2.0 - http://www.apache.org/licens Review Comment: Done -- T

Re: [I] Support pushdown filters for non-cast date conversion functions (e.g. to_date) [iceberg-rust]

2025-02-02 Thread via GitHub
kevinjqliu commented on issue #933: URL: https://github.com/apache/iceberg-rust/issues/933#issuecomment-2629482732 Thanks for creating this issue @omerhadari. Is this something you would like to contribute? As we discussed on slack, Fokko added something similar in #821 for cast operati

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938536328 ## gcp-bundle/LICENSE: ## @@ -325,24 +315,24 @@ License: The Apache Software License, Version 2.0 - http://www.apache.org/licens Review Comment: I agree. Let

Re: [PR] Fix NOTICE and LICENSE in the gcp-bundle jar [iceberg]

2025-02-02 Thread via GitHub
amogh-jahagirdar commented on code in PR #12144: URL: https://github.com/apache/iceberg/pull/12144#discussion_r1938533960 ## gcp-bundle/LICENSE: ## @@ -325,24 +315,24 @@ License: The Apache Software License, Version 2.0 - http://www.apache.org/licens Review Comment: Sure,

Re: [PR] Add data type/schema field/schema [iceberg-cpp]

2025-02-02 Thread via GitHub
zhjwpku commented on PR #31: URL: https://github.com/apache/iceberg-cpp/pull/31#issuecomment-2629416131 > Is anything else required here? +1 to merge this PR, thanks for working on this. -- This is an automated message from the Apache Git Service. To respond to the message, please l

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
RussellSpitzer commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938500032 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -355,14 +318,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0 --

Re: [PR] feat: Support metadata table "Entries" [iceberg-rust]

2025-02-02 Thread via GitHub
xxchan commented on PR #863: URL: https://github.com/apache/iceberg-rust/pull/863#issuecomment-2629401449 > `MapBuilder` doesn't let you configure the key field though, so the keys array won't have the field id. Not sure if I understand it correctly, this sounds like just some limita

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
RussellSpitzer commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938492198 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -219,54 +219,17 @@ License: http://www.apache.org/licenses/LICENSE-2.0.txt -

[PR] chore(deps): Bump aws-sdk-s3tables from 1.7.0 to 1.8.0 [iceberg-rust]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #938: URL: https://github.com/apache/iceberg-rust/pull/938 Bumps [aws-sdk-s3tables](https://github.com/awslabs/aws-sdk-rust) from 1.7.0 to 1.8.0. Commits See full diff in https://github.com/awslabs/aws-sdk-rust/commits";>compare view

[PR] chore(deps): Bump parquet from 53.3.0 to 53.4.0 [iceberg-rust]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #935: URL: https://github.com/apache/iceberg-rust/pull/935 Bumps [parquet](https://github.com/apache/arrow-rs) from 53.3.0 to 53.4.0. Release notes Sourced from https://github.com/apache/arrow-rs/releases";>parquet's releases. 53.4.

[PR] chore(deps): Bump arrow-string from 53.3.0 to 53.4.0 [iceberg-rust]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #937: URL: https://github.com/apache/iceberg-rust/pull/937 Bumps [arrow-string](https://github.com/apache/arrow-rs) from 53.3.0 to 53.4.0. Release notes Sourced from https://github.com/apache/arrow-rs/releases";>arrow-string's releases.

[PR] chore(deps): Bump async-trait from 0.1.85 to 0.1.86 [iceberg-rust]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #936: URL: https://github.com/apache/iceberg-rust/pull/936 Bumps [async-trait](https://github.com/dtolnay/async-trait) from 0.1.85 to 0.1.86. Release notes Sourced from https://github.com/dtolnay/async-trait/releases";>async-trait's relea

Re: [I] Variant Data Type Support [iceberg]

2025-02-02 Thread via GitHub
desairavi1101 commented on issue #10392: URL: https://github.com/apache/iceberg/issues/10392#issuecomment-2629380475 +1 -- 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 u

Re: [PR] feat: Support metadata table "Entries" [iceberg-rust]

2025-02-02 Thread via GitHub
rshkv commented on PR #863: URL: https://github.com/apache/iceberg-rust/pull/863#issuecomment-2629378449 @xxchan, that's right. The problem is constructing Array/RecordBatch with the right schema in the first place. `MapBuilder` doesn't let you configure the key field though, so the keys ar

[PR] chore(deps): Bump crate-ci/typos from 1.29.4 to 1.29.5 [iceberg-rust]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #934: URL: https://github.com/apache/iceberg-rust/pull/934 Bumps [crate-ci/typos](https://github.com/crate-ci/typos) from 1.29.4 to 1.29.5. Release notes Sourced from https://github.com/crate-ci/typos/releases";>crate-ci/typos's releases.

Re: [PR] Add data type/schema field/schema [iceberg-cpp]

2025-02-02 Thread via GitHub
lidavidm commented on PR #31: URL: https://github.com/apache/iceberg-cpp/pull/31#issuecomment-2629358646 Is anything else required 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 specif

Re: [I] Spark: Add read/write support for UUIDs from bytes [iceberg]

2025-02-02 Thread via GitHub
RussellSpitzer commented on issue #10635: URL: https://github.com/apache/iceberg/issues/10635#issuecomment-2629355428 @anuragmantri are you still interested in picking this one up? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459556 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -456,37 +387,6 @@ License text: -Th

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459592 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -519,30 +419,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459460 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -403,22 +350,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938460221 ## spark/v3.5/spark-runtime/NOTICE: ## @@ -500,9 +363,52 @@ file: This binary artifact includes Project Nessie with the following in its NOTICE file: +| NOTICE a

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938460072 ## spark/v3.5/spark-runtime/NOTICE: ## @@ -82,41 +69,6 @@ NOTICE file: -This b

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938460128 ## spark/v3.5/spark-runtime/NOTICE: ## @@ -82,41 +69,6 @@ NOTICE file: -This b

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938460015 ## spark/v3.5/spark-runtime/NOTICE: ## @@ -26,19 +26,6 @@ the following copyright notice: ---

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459504 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -403,22 +350,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459394 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -379,14 +334,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459210 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -355,14 +318,6 @@ License: http://www.apache.org/licenses/LICENSE-2.0

Re: [PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre commented on code in PR #12160: URL: https://github.com/apache/iceberg/pull/12160#discussion_r1938459131 ## spark/v3.5/spark-runtime/LICENSE: ## @@ -219,54 +219,17 @@ License: http://www.apache.org/licenses/LICENSE-2.0.txt ---

[PR] Fix NOTICE and LICENSE in the spark-runtime jar [iceberg]

2025-02-02 Thread via GitHub
jbonofre opened a new pull request, #12160: URL: https://github.com/apache/iceberg/pull/12160 This PR does: * Update `Avro` copyright * Remove paranamer (not found in the jar) * Update `Parquet` copyright * Update `Thrift` copyright * Update `ORC` copyright * Update `Hive` c

Re: [I] Spark: Add read/write support for UUIDs from bytes [iceberg]

2025-02-02 Thread via GitHub
raphaelauv commented on issue #10635: URL: https://github.com/apache/iceberg/issues/10635#issuecomment-2629319419 No stale -- 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.

[PR] Retry on NoSuchNamespaceException not found in rename table [iceberg]

2025-02-02 Thread via GitHub
huan233usc opened a new pull request, #12159: URL: https://github.com/apache/iceberg/pull/12159 This PR attempts to fix the issue mentioned in https://github.com/apache/iceberg/issues/11154. On NoSuchNamespaceException, we retry rename table with stripping the catalog name if the fir

Re: [I] REST Catalog does not validate "to" identifier on rename table [iceberg]

2025-02-02 Thread via GitHub
huan233usc commented on issue #11154: URL: https://github.com/apache/iceberg/issues/11154#issuecomment-2629305631 One option to fix the issue might be retry on NoSuchNamespaceException with stripping catalog name? Given rest catalog supports multi-level name space, we can't tell top level i

Re: [PR] Build: Bump me.champeau.jmh:jmh-gradle-plugin from 0.7.2 to 0.7.3 [iceberg]

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

Re: [PR] Build: Bump net.snowflake:snowflake-jdbc from 3.21.0 to 3.22.0 [iceberg]

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

Re: [PR] Build: Bump software.amazon.awssdk:bom from 2.30.6 to 2.30.11 [iceberg]

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

[PR] build(deps): bump github.com/uptrace/bun/dialect/mysqldialect from 1.2.8 to 1.2.9 [iceberg-go]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #284: URL: https://github.com/apache/iceberg-go/pull/284 Bumps [github.com/uptrace/bun/dialect/mysqldialect](https://github.com/uptrace/bun) from 1.2.8 to 1.2.9. Release notes Sourced from https://github.com/uptrace/bun/releases";>github.

[PR] build(deps): bump github.com/uptrace/bun/dialect/mssqldialect from 1.2.8 to 1.2.9 [iceberg-go]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #285: URL: https://github.com/apache/iceberg-go/pull/285 Bumps [github.com/uptrace/bun/dialect/mssqldialect](https://github.com/uptrace/bun) from 1.2.8 to 1.2.9. Release notes Sourced from https://github.com/uptrace/bun/releases";>github.

[PR] build(deps): bump github.com/uptrace/bun/dialect/pgdialect from 1.2.8 to 1.2.9 [iceberg-go]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #283: URL: https://github.com/apache/iceberg-go/pull/283 Bumps [github.com/uptrace/bun/dialect/pgdialect](https://github.com/uptrace/bun) from 1.2.8 to 1.2.9. Release notes Sourced from https://github.com/uptrace/bun/releases";>github.com

[PR] build(deps): bump github.com/uptrace/bun/dialect/sqlitedialect from 1.2.8 to 1.2.9 [iceberg-go]

2025-02-02 Thread via GitHub
dependabot[bot] opened a new pull request, #282: URL: https://github.com/apache/iceberg-go/pull/282 Bumps [github.com/uptrace/bun/dialect/sqlitedialect](https://github.com/uptrace/bun) from 1.2.8 to 1.2.9. Release notes Sourced from https://github.com/uptrace/bun/releases";>github

  1   2   >