canyang25 commented on code in PR #59298:
URL: https://github.com/apache/spark/pull/59298#discussion_r4239809704


##########
python/pyspark/sql/conversion.py:
##########
@@ -1429,7 +1430,20 @@ def convert_timestamp(value: Any) -> Any:
                     # lossless path), so drop any sub-microsecond digits to 
match classic collect().
                     if hasattr(value, "to_pydatetime"):
                         value = value.to_pydatetime(warn=False)
-                    return value.astimezone().replace(tzinfo=None)
+                    # SPARK-60081: integer POSIX seconds so fromtimestamp sets 
fold on the
+                    # local wall time, then re-attach microseconds. A float 
timestamp would
+                    # drop microseconds past ~2255 and can overflow at 
9999-12-31 under UTC.

Review Comment:
   Fixed, it now says ~2242.



-- 
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]

Reply via email to