dilipbiswal commented on a change in pull request #28593:
URL: https://github.com/apache/spark/pull/28593#discussion_r436519100
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala
##########
@@ -2595,6 +2595,15 @@ object SQLConf {
.checkValue(_ > 0, "The timeout value must be positive")
.createWithDefault(10L)
+ val LEGACY_ALLOW_CAST_NUMERIC_TO_TIMESTAMP =
+ buildConf("spark.sql.legacy.allowCastNumericToTimestamp")
+ .internal()
+ .doc("When true, allow cast numeric to timestamp, but for integral
numbers," +
+ "Hive treats it as milliseconds, Spark SQL treats n as seconds")
Review comment:
Do we need to mention hive here ? Can we not just state the spark
behavior ? Secondly not sure what is N here ?
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
