rdblue commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1768939469
##########
format/spec.md:
##########
@@ -200,12 +200,15 @@ Supported primitive types are defined in the table below.
Primitive types added
| | **`uuid`** | Universally unique identifiers
| Should use 16-byte fixed
|
| | **`fixed(L)`** | Fixed-length byte array of length L
|
|
| | **`binary`** | Arbitrary-length byte array
|
|
+| [v3](#version-3) | **`geometry(C, E)`** | An object of the simple feature
geometry model as defined by Appendix G; This may be any of the geometry
subclasses defined therein; crs C [4], edges E [5] | C and E are fixed, and if
unset will take default values. |
Review Comment:
I think we can be more specific here and call out the standard that we are
referencing, like we do with IEEE 754. This is "Geometry features as WKB(link)
stored in coordinate reference system C and edge type E (see Appendix G)"
I would also say that "If not specified, C is OGC:CRS84 and E is planar".
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
