paleolimbot commented on code in PR #45459:
URL: https://github.com/apache/arrow/pull/45459#discussion_r2017853950
##########
cpp/src/parquet/CMakeLists.txt:
##########
@@ -171,6 +171,9 @@ set(PARQUET_SRCS
exception.cc
file_reader.cc
file_writer.cc
+ geospatial_statistics.cc
Review Comment:
The `parquet/` directory doesn't currently use subdirectories for other
files with related functionality (e.g., there are more `level_` files than
`geospatial_` ones), although I'm happy to move them anywhere!
Perhaps a bigger question, but I hope that there are not any more geo data
utilities coming to Parquet (if anything, I'd like to figure out how to move
some of these things out of Parquet using a canonical extension type or compute
functions registered at runtime).
--
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: github-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
