wgtmac commented on code in PR #18:
URL: https://github.com/apache/orc-format/pull/18#discussion_r2034293559


##########
src/main/proto/orc/proto/orc_proto.proto:
##########
@@ -84,6 +84,27 @@ message CollectionStatistics {
   optional uint64 total_children = 3;
 }
 
+// Bounding box for Geometry or Geography type in the representation of min/max
+// value pair of coordinates from each axis.
+message BoundingBox {
+  optional double xmin = 1;
+  optional double xmax = 2;
+  optional double ymin = 3;
+  optional double ymax = 4;
+  optional double zmin = 5;
+  optional double zmax = 6;
+  optional double mmin = 7;
+  optional double mmax = 8;
+}
+
+// Statistics specific to Geometry or Geography type
+message GeospatialStatistics {
+  // A bounding box of geospatial instances
+  optional BoundingBox bbox = 1;

Review Comment:
   Yes, `bbox` is a well-known geospatial acronym.  



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to