uros-b commented on code in PR #56422:
URL: https://github.com/apache/spark/pull/56422#discussion_r3903290006
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -2082,6 +2082,16 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val PARQUET_TIME_TYPE_ALLOW_IS_ADJUSTED_TO_UTC_READ =
+ buildConf("spark.sql.parquet.timeType.allowIsAdjustedToUtcRead")
+ .doc("When true, Spark reads Parquet TIME(MICROS,isAdjustedToUTC=true)
as TimeType, " +
+ "for compatibility with writers such as Apache Arrow. When false
(default), reading " +
+ "TIME(MICROS,isAdjustedToUTC=true) throws an error.")
Review Comment:
The config contract seems a bit inaccurate. Here it says reads fail when
disabled, but the flag only gates physical-schema inference. Explicit TimeType
schemas already succeed without it, as the PR’s test demonstrates.
--
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]
