Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21169#discussion_r184731765
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
    @@ -1208,6 +1208,13 @@ object SQLConf {
           .stringConf
           .createWithDefault("")
     
    +  val REJECT_TIMEZONE_IN_STRING = 
buildConf("spark.sql.function.rejectTimezoneInString")
    +    .internal()
    +    .doc("If true, `to_utc_timestamp` and `from_utc_timestamp` return null 
if the input string " +
    +      "contains a timezone part, e.g. `2000-10-10 00:00:00+00:00`.")
    +    .booleanConf
    +    .createWithDefault(true)
    +
    --- End diff --
    
    existing workloads may depend on the previous behavior and think that's 
corrected. It's safer to provide an internal conf and tell users about it when 
they complain about behavior change. It's an internal conf and is invisible to 
end users. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to