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


##########
format/spec.md:
##########
@@ -190,6 +190,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, T, E)`** | An object of the simple feature 
geometry model as defined by Appendix G; This may be any of the Geometry 
subclasses defined therein; coordinate reference system C [4], coordinate 
reference system type T [5], edges E [6] | C, T, E are fixed. Encoded as WKB, 
see Appendix G.   |

Review Comment:
   What syntax to use for an engine to create the geometry type? Does it 
require C/T/E to appear in the type?



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