wgtmac commented on PR #240: URL: https://github.com/apache/parquet-format/pull/240#issuecomment-2271499591
> We should remove S2 / H3 from the covering column because it is still unclear about how to create statistics for them. Make sense. Let me remove it. > We should specify the ordering of coordinates in the bbox covering. Did you mean removing the `struct BoundingBox` and using the `struct Covering` to represent the bbox: https://github.com/apache/parquet-format/pull/240/files#diff-834c5a8d91719350b20995ad99d1cb6d8d68332b9ac35694f40e375bdb2d3e7cR268?? In this way, the WKB encoding will handle the coordinates, right? @jiayuasu > it should be clear how this is going to get encoded as binary (JSON string? Little-endian doubles with specific ordering?). IIUC, the `Covering/kind` field has already specified the way to encode the data. For example, WKB for bbox. @paleolimbot > I think we don't need both WKT and WKB (just WKB would be my preference!) Yes, WKB is always preferred in this case. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
