cashmand commented on code in PR #50270:
URL: https://github.com/apache/spark/pull/50270#discussion_r2001753869
##########
common/variant/src/main/java/org/apache/spark/types/variant/VariantUtil.java:
##########
@@ -123,6 +123,12 @@ public class VariantUtil {
// Long string value. The content is (4-byte little-endian unsigned integer
representing the
// string size) + (size bytes of string content).
public static final int LONG_STR = 16;
+ // Similar to TIMESTAMP, but the 8-byte value represents the number of
nanoseconds since the
Review Comment:
@rdblue pointed out to me that the range is documented in the Parquet spec:
https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#:~:text=Due%20to%20the,practical%20everyday%20usage.
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
