Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis merged PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855 -- 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: commits-unsubscr...@

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2139384658 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2139381407 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2139377617 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2139225667 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2139219914 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619450379 ## baremaps-core/src/main/java/org/apache/baremaps/storage/geoparquet/GeoParquetTypeConversion.java: ## @@ -0,0 +1,77 @@ +/* + * License

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2138223634 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2138219247 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2137782209 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2137782511 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2137770551 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619143177 ## baremaps-core/src/test/java/org/apache/baremaps/storage/geoparquet/GeoParquetToPostgresTest.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619142230 ## baremaps-core/src/main/java/org/apache/baremaps/storage/geoparquet/GeoParquetDataSchema.java: ## @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache Software F

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2137762249 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619135943 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619135943 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619135943 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619135943 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2137724373 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619110238 ## baremaps-core/src/main/java/org/apache/baremaps/storage/geoparquet/GeoParquetTypeConversion.java: ## @@ -0,0 +1,77 @@ +/* + * Licensed to the Apache Softw

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619089065 ## baremaps-core/src/test/java/org/apache/baremaps/storage/geoparquet/GeoParquetToPostgresTest.java: ## @@ -0,0 +1,50 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619084887 ## baremaps-core/src/main/java/org/apache/baremaps/storage/geoparquet/GeoParquetTypeConversion.java: ## @@ -0,0 +1,77 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1619084887 ## baremaps-core/src/main/java/org/apache/baremaps/storage/geoparquet/GeoParquetTypeConversion.java: ## @@ -0,0 +1,77 @@ +/* + * Licensed to the Apache Softwa

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1618553524 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,277 @@ +/* + * Licensed to the

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2136953944 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1617937771 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,249 @@ +/* + * Licensed to the

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2136174530 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2133966695 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2133964999 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2133958623 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2133936755 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2131242523 I did a little bit of reading on the spec of geoparquet. https://github.com/opengeospatial/geoparquet/blob/main/format-specs/geoparquet.md I have a feeling that in our l

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1614617900 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/hadoop/GeoParquetGroupWriteSupport.java: ## Review Comment: Yes, I kept them with th

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1614608962 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/hadoop/GeoParquetGroupWriteSupport.java: ## Review Comment: All the writer files are

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2130318794 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613994827 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613993646 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Found

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613992792 ## pom.xml: ## @@ -321,6 +340,16 @@ limitations under the License. lucene-spatial-extras ${version.lib.lucene} + Review Comm

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613989180 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,184 @@ +/* + * Licensed to the

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2130297886 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2130295171 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613713190 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/data/FileInfo.java: ## Review Comment: We can probably discuss organising these files

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613144814 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613144814 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613107467 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613107467 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,263 @@ +/* + * Licensed to the Apache Software Founda

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613104762 ## baremaps-geoparquet/pom.xml: ## @@ -0,0 +1,54 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; xsi:schema

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613093655 ## pom.xml: ## @@ -321,6 +340,16 @@ limitations under the License. lucene-spatial-extras ${version.lib.lucene} + Review Comme

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1613085030 ## pom.xml: ## @@ -95,13 +96,15 @@ limitations under the License. 6.6.4 23.0.3 33.0.0-jre +3.4.0 5.1.0 1.52 2.16.1 1.19

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2126406044 @sebr72 I just pushed the `GeoParquetGroup` interface, which I think would be a decent abstraction to access the records of a geoparquet file. It contains getters and setters f

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2126397664 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1611079446 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/data/GeoParquetGroupImpl.java: ## @@ -0,0 +1,284 @@ +/* + * Licensed

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2126331152 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2126330459 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124906010 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124905691 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124903383 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1610010156 ## baremaps-geoparquet/src/test/java/org/apache/baremaps/geoparquet/GeoParquetReaderTest.java: ## @@ -0,0 +1,42 @@ +/* + * Licensed to t

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124855799 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124202443 Previous attempt #851 -- 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: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1609543001 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/data/GeoParquetGroupConverter.java: ## @@ -0,0 +1,69 @@ +/* + * Lice

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124191651 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2124185876 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1609531753 ## pom.xml: ## @@ -95,13 +96,15 @@ limitations under the License. 6.6.4 23.0.3 33.0.0-jre +3.4.0 5.1.0 1.52 2.16.1 1.1

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1609526193 ## baremaps-geoparquet/pom.xml: ## @@ -0,0 +1,46 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; xsi:schem

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

github-advanced-security[bot] commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1609042961 ## baremaps-geoparquet/src/main/java/org/apache/baremaps/geoparquet/GeoParquetReader.java: ## @@ -0,0 +1,289 @@ +/* + * Licensed to the

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2123571420 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

Drabble commented on code in PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#discussion_r1608989495 ## baremaps-geoparquet/pom.xml: ## @@ -0,0 +1,46 @@ + +http://maven.apache.org/POM/4.0.0"; xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; xsi:schema

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2123445413 ## [](https:

Re: [PR] Create a geoparquet module and add dependencies [incubator-baremaps]

sonarcloud[bot] commented on PR #855: URL: https://github.com/apache/incubator-baremaps/pull/855#issuecomment-2123435583 ## [](https:

[PR] Create a geoparquet module and add dependencies [incubator-baremaps]

bchapuis opened a new pull request, #855: URL: https://github.com/apache/incubator-baremaps/pull/855 (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 unsubsc