jiayuasu commented on code in PR #494:
URL: https://github.com/apache/parquet-format/pull/494#discussion_r2059723301


##########
Geospatial.md:
##########
@@ -162,3 +192,18 @@ The axis order of the coordinates in WKB and bounding box 
stored in Parquet
 follows the de facto standard for axis order in WKB and is therefore always
 (x, y) where x is easting or longitude and y is northing or latitude. This
 ordering explicitly overrides the axis order as specified in the CRS.
+
+# Invalid geospatial values
+
+An invalid geospatial value refers to any of the following cases:
+
+* `null`: A null value in Parquet.
+* A non-`null` value that are encoded in a valid WKB or bounding box format 
+  but are not considered valid under this specification, including:
+  * `NaN`: Not a Number. For example, `POINT EMPTY` in WKB is represented by a 
+    `Point` with each ordinate value set to an IEEE-754 quiet NaN value.
+  * `Empty geometries`: Geometries explicitly marked as empty in WKB using 

Review Comment:
   This is really a WKB reader-specific behavior. I added the following line:
   
   > While different WKB readers may interpret such values differently, the 
resulting output should be treated as invalid



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

Reply via email to