emkornfield commented on code in PR #496:
URL: https://github.com/apache/parquet-format/pull/496#discussion_r2231681749


##########
LogicalTypes.md:
##########
@@ -539,6 +544,27 @@ The sort order used for `INTERVAL` is undefined. When 
writing data, no min/max
 statistics should be saved for this type and if such non-compliant statistics
 are found during reading, they must be ignored.
 
+#### YEAR_MONTH_INTERVAL
+
+`YEAR_MONTH_INTERVAL` is used to represent a year-month time interval, such as
+`4 years and 6 months`. It must annotate an `int32` that stores the total 
number
+of months as a signed integer, which represents the interval and can be 
negative.

Review Comment:
   ```suggestion
   of months as a signed integer, which represents the interval and can be 
negative. Some ANSI SQL systems only support intervals in the range of +/- 
10,000 years, no constraint is placed on the exact value in Parquet.
   ```



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