Re: [PR] PARQUET-2417: Add support for geometry logical type [parquet-java]

2025-02-27 Thread via GitHub
jiayuasu commented on code in PR #2971: URL: https://github.com/apache/parquet-java/pull/2971#discussion_r1974625631 ## parquet-column/src/main/java/org/apache/parquet/column/statistics/geometry/BoundingBox.java: ## @@ -0,0 +1,203 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] GH-473: Add shredding version [parquet-format]

2025-02-27 Thread via GitHub
wgtmac commented on PR #474: URL: https://github.com/apache/parquet-format/pull/474#issuecomment-2689851641 What is the progress of this? Should we merge or close it in order to release the parquet-format 2.11.0? @emkornfield @rdblue -- This is an automated message from the Apache Git Se

Re: [PR] Example files for GEOMETRY and GEOGRAPHY logical type [parquet-testing]

2025-02-27 Thread via GitHub
paleolimbot commented on PR #70: URL: https://github.com/apache/parquet-testing/pull/70#issuecomment-2689296707 I pushed an update to three files here - the original fields that PROJJSON crses were written to were very likely to collide with eachother if you did things like read a Parquet f

Re: [PR] Add logical type annotation for `UnknownType` [parquet-java]

2025-02-27 Thread via GitHub
Fokko merged PR #3154: URL: https://github.com/apache/parquet-java/pull/3154 -- 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...@parquet.

Re: [PR] PARQUET-2417: Add support for geometry logical type [parquet-java]

2025-02-27 Thread via GitHub
jiayuasu commented on code in PR #2971: URL: https://github.com/apache/parquet-java/pull/2971#discussion_r1974097402 ## parquet-column/src/main/java/org/apache/parquet/column/statistics/geometry/BoundingBox.java: ## @@ -0,0 +1,203 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] PARQUET-2417: Add support for geometry logical type [parquet-java]

2025-02-27 Thread via GitHub
paleolimbot commented on code in PR #2971: URL: https://github.com/apache/parquet-java/pull/2971#discussion_r1974432299 ## parquet-column/src/main/java/org/apache/parquet/column/statistics/geometry/BoundingBox.java: ## @@ -0,0 +1,203 @@ +/* + * Licensed to the Apache Software Fo

Re: [PR] PARQUET-2417: Add support for geometry logical type [parquet-java]

2025-02-27 Thread via GitHub
paleolimbot commented on code in PR #2971: URL: https://github.com/apache/parquet-java/pull/2971#discussion_r1974584715 ## parquet-column/src/main/java/org/apache/parquet/column/statistics/geometry/BoundingBox.java: ## @@ -0,0 +1,203 @@ +/* + * Licensed to the Apache Software Fo

Re: [PR] Add logical type annotation for `UnknownType` [parquet-java]

2025-02-27 Thread via GitHub
Fokko commented on PR #3154: URL: https://github.com/apache/parquet-java/pull/3154#issuecomment-2688954311 @wgtmac Thanks. I'm not a big fan of the issues (I keep forgetting to create one), but I do think it would be good to add more context to the PR itself. -- This is an automated messa