paleolimbot commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1882340902


##########
format/spec.md:
##########
@@ -205,13 +205,18 @@ 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)`**  | Geometry features from [OGC – Simple 
feature access](https://portal.ogc.org/files/?artifact_id=25355). Edges 
interpolation is always linear/planar. See [Appendix 
G](#appendix-g-geospatial-notes). Parameterized by crs C [4]. If not specified, 
C is `OGC:CRS84`. |                                                        |

Review Comment:
   I happen to think either is fine..."planar" has precedent because of the 
simple features access standard; linear is more correct. I don't think there's 
any ambiguity in either case and am pretty happy with anything here.



-- 
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

Reply via email to