rdblue commented on code in PR #13227:
URL: https://github.com/apache/iceberg/pull/13227#discussion_r2125045399
##########
format/spec.md:
##########
@@ -221,7 +221,7 @@ 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)`** | Geospatial features from [OGC –
Simple feature access][1001]. Edge-interpolation is always linear/planar. See
[Appendix G](#appendix-g-geospatial-notes). Parameterized by CRS C. If not
specified, C is `OGC:CRS84`. |
|
+| [v3](#version-3) | **`geometry(C)`** | Geospatial features from [OGC –
Simple feature access][1001]. Edge-interpolation is always linear/planar, i.e.,
geometric calculations are performed using Cartesian mathematics along straight
lines. See [Appendix G](#appendix-g-geospatial-notes). Parameterized by CRS C.
If not specified, C is `OGC:CRS84`. |
|
Review Comment:
Should we mention that the purpose of Cartesian mathematics is to avoid the
need to interpret the CRS?
Something like:
> The CRS may be used to interpret points as geographic locations, but does
not affect calculations, which are always Cartesian.
--
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]
