paleolimbot commented on code in PR #45459:
URL: https://github.com/apache/arrow/pull/45459#discussion_r2059543202
##########
cpp/src/parquet/types.cc:
##########
@@ -1630,6 +1669,217 @@ class LogicalType::Impl::Float16 final : public
LogicalType::Impl::Incompatible,
GENERATE_MAKE(Float16)
+namespace {
+std::string EscapeControl(char c) {
Review Comment:
Since we have RapidJSON as required now, I updated this to use RapidJSON to
sanitize the CRS as a string.
##########
python/pyarrow/_parquet.pyx:
##########
@@ -319,6 +319,136 @@ cdef _box_flba(ParquetFLBA val, uint32_t len):
return cp.PyBytes_FromStringAndSize(<char*> val.ptr, <Py_ssize_t> len)
+cdef class GeoStatistics(_Weakrefable):
+ """Statistics for columns with geospatial data types (experimental)"""
+
+ def __init__(self):
+ raise TypeError(f"Do not call {self.__class__.__name__}'s constructor
directly")
+
+ def __cinit__(self):
+ pass
+
+ def __repr__(self):
+ return f"""{object.__repr__(self)}
+ geospatial_types: {self.geospatial_types}
+ xmin: {self.xmin}, xmax: {self.xmax}
+ ymin: {self.ymin}, ymax: {self.ymax}
+ zmin: {self.zmin}, zmax: {self.zmax}
+ mmin: {self.mmin}, mmax: {self.mmax}"""
+
+ def to_dict(self):
+ out = {
+ "geospatial_types": self.geospatial_types,
+ "xmin": self.xmin,
+ "xmax": self.xmax,
+ "ymin": self.ymin,
+ "ymax": self.ymax,
+ "zmin": self.zmin,
+ "zmax": self.zmax,
+ "mmin": self.mmin,
+ "mmax": self.mmax
+ }
+
+ return out
+
+ @property
+ def geospatial_types(self):
Review Comment:
Done!
--
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
