MaxGekk commented on issue #23811: [SPARK-26902][SQL] Support java.time.Instant as an external type of TimestampType URL: https://github.com/apache/spark/pull/23811#issuecomment-578394207 > How useful is this change? Please, take a look at the motivation points in the PR description. > Wouldn't it break a lot of user code that use Timestmap type when they upgrade to 3.0? No, it will not break because Spark still returns `java.sql.Timestamp` by default. > It seems like we wouldn't be able to ever remove the config flag. The flag has been removed already, and replaced by `spark.sql.datetime.java8API.enabled` (see https://github.com/apache/spark/blob/ed44926117d81aa5fa8bd823d401efd235260872/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala#L2033-L2039) which is disabled by default.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
