paleolimbot commented on code in PR #45459:
URL: https://github.com/apache/arrow/pull/45459#discussion_r2059088194
##########
cpp/src/parquet/column_writer.cc:
##########
@@ -1826,6 +1870,12 @@ Status
TypedColumnWriterImpl<ParquetType>::WriteArrowDictionary(
page_statistics_->IncrementNullCount(num_chunk_levels - non_null_count);
page_statistics_->IncrementNumValues(non_null_count);
page_statistics_->Update(*referenced_dictionary, /*update_counts=*/false);
+
+ if (chunk_geospatial_statistics_ != nullptr) {
+ throw ParquetException(
+ "Writing dictionary-encoded GEOMETRY or GEOGRAPHY with statistics is
not "
+ "supported");
+ }
Review Comment:
I'm not sure if this will ever come up on purpose...it's definitely not a
typical use case to store a lot of repeated geometries. If this happens by
accident/automatically for a reason I haven't learned about yet I'm happy to
circle back.
--
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
