MaxGekk commented on code in PR #56853:
URL: https://github.com/apache/spark/pull/56853#discussion_r3489852593
##########
python/pyspark/sql/functions/builtin.py:
##########
@@ -11709,7 +11709,7 @@ def unix_date(col: "ColumnOrName") -> Column:
@_try_remote_functions
def unix_micros(col: "ColumnOrName") -> Column:
- """Returns the number of microseconds since 1970-01-01 00:00:00 UTC.
+ """Returns the number of microseconds since 1970-01-01 00:00:00 UTC.
Truncates higher levels of precision.
Review Comment:
Appending the sentence here makes this line 110 characters, over Spark's
100-char Python limit (`dev/tox.ini` flake8 `max-line-length = 100`;
`pyproject.toml` black/ruff `line-length = 100`), so it'll fail the lint CI
gate.
It also doesn't quite match `unix_millis` (line 11754) and `unix_seconds`
(line 11848) — the goal of this PR — since both put the disclaimer on its **own
line**. Moving it to a separate line fixes both:
```suggestion
"""Returns the number of microseconds since 1970-01-01 00:00:00 UTC.
Truncates higher levels of precision.
```
--
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]
