This is an automated email from the ASF dual-hosted git repository.
ruifengz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 75cb479c9510 [SPARK-53112][SQL][PYTHON][CONNECT][FOLLOW-UP] Change
date and time to be positional arguments
75cb479c9510 is described below
commit 75cb479c9510c6eaf97861c862a2c365c79ee5a0
Author: Yicong-Huang <[email protected]>
AuthorDate: Fri Sep 26 08:26:41 2025 +0800
[SPARK-53112][SQL][PYTHON][CONNECT][FOLLOW-UP] Change date and time to be
positional arguments
### What changes were proposed in this pull request?
Use positional arguments for `date` and `time` fields in
`make_timestamp_ntz` and `try_make_timestamp_ntz`.
### Why are the changes needed?
To fix the unnecessary requirement of them being keyword arguments.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Unit tests in place.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #52461 from Yicong-Huang/SPARK-53112/fix/use-positional-arguments.
Authored-by: Yicong-Huang <[email protected]>
Signed-off-by: Ruifeng Zheng <[email protected]>
---
python/pyspark/sql/connect/functions/builtin.py | 2 --
python/pyspark/sql/functions/builtin.py | 2 --
2 files changed, 4 deletions(-)
diff --git a/python/pyspark/sql/connect/functions/builtin.py
b/python/pyspark/sql/connect/functions/builtin.py
index 408e43f1a62c..391469765f62 100644
--- a/python/pyspark/sql/connect/functions/builtin.py
+++ b/python/pyspark/sql/connect/functions/builtin.py
@@ -4047,7 +4047,6 @@ def make_timestamp_ntz(
hours: Optional["ColumnOrName"] = None,
mins: Optional["ColumnOrName"] = None,
secs: Optional["ColumnOrName"] = None,
- *,
date: Optional["ColumnOrName"] = None,
time: Optional["ColumnOrName"] = None,
) -> Column:
@@ -4108,7 +4107,6 @@ def try_make_timestamp_ntz(
hours: Optional["ColumnOrName"] = None,
mins: Optional["ColumnOrName"] = None,
secs: Optional["ColumnOrName"] = None,
- *,
date: Optional["ColumnOrName"] = None,
time: Optional["ColumnOrName"] = None,
) -> Column:
diff --git a/python/pyspark/sql/functions/builtin.py
b/python/pyspark/sql/functions/builtin.py
index 1df8ddd1f432..cf54fd23e818 100644
--- a/python/pyspark/sql/functions/builtin.py
+++ b/python/pyspark/sql/functions/builtin.py
@@ -25194,7 +25194,6 @@ def make_timestamp_ntz(
hours: Optional["ColumnOrName"] = None,
mins: Optional["ColumnOrName"] = None,
secs: Optional["ColumnOrName"] = None,
- *,
date: Optional["ColumnOrName"] = None,
time: Optional["ColumnOrName"] = None,
) -> Column:
@@ -25352,7 +25351,6 @@ def try_make_timestamp_ntz(
hours: Optional["ColumnOrName"] = None,
mins: Optional["ColumnOrName"] = None,
secs: Optional["ColumnOrName"] = None,
- *,
date: Optional["ColumnOrName"] = None,
time: Optional["ColumnOrName"] = None,
) -> Column:
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
