Github user httfighter commented on the issue:

    https://github.com/apache/spark/pull/19380
  
    I understand everyone's worries.But i hava few thoughts.
    Firstly, the native unix_timestamp itself supports the "yyyy-MM-dd 
HH:mm:ss.SSS" form of the date, but the result is lost in milliseconds when i 
use it. Obviously, it's a bug. It will give users the wrong results
    I think this should be fixed.
    Secondly,  unix_timestamp, from_unixtime and to_unix_timestamp all have the 
similar bug, but related methods only these three.  I think the data type of 
unix time of the three methods should be defined as DoubleType,not for 
LongType. Or in milliseconds which will bring more problems. 


---

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

Reply via email to