lidavidm commented on code in PR #496:
URL: https://github.com/apache/parquet-format/pull/496#discussion_r2156159721
##########
src/main/thrift/parquet.thrift:
##########
@@ -461,6 +461,29 @@ struct GeographyType {
2: optional EdgeInterpolationAlgorithm algorithm;
}
+/**
+ * Year-Month Interval logical type annotation
+ *
+ * The data is stored as an 4 byte signed integer which represents the number
+ * of months associated with the time interval. The value can be negative to
+ * indicate a backward duration.
+ *
+ * Allowed for physical type: INT32
+ */
+struct IntervalYearMonthType {
+}
+
+/**
+ * Month-Day Interval logical type annotation
+ *
+ * The data is stored as a 16-byte signed value, which represents the number
Review Comment:
Ok, got it. I suppose the strictly closest Arrow type, if there were one,
would be 128 bit duration then (and I suppose we would want to define int128
while we're at it).
I think if we're going to do this, we should kick off a parallel discussion
in Arrow (and maybe RAPIDS/cuDF) for a potential new (extension) type since
otherwise we're won't be able to actually represent the new type in (for
instance) all our client libraries.
--
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]
