Skip to content

Conversation

@dongjoon-hyun
Copy link
Member

@dongjoon-hyun dongjoon-hyun commented Dec 11, 2025

What changes were proposed in this pull request?

This PR aims to add versionadded to TimeType.

Why are the changes needed?

For Scala/Java documentation, we have @since information.

* @since 4.1.0
*/
@Unstable
case class TimeType(precision: Int) extends AnyTimeType {

We had better provide a correct information for the newly added TimeType in Python documentation.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass the CIs.

Was this patch authored or co-authored using generative AI tooling?

No.

@dongjoon-hyun
Copy link
Member Author

@dongjoon-hyun
Copy link
Member Author

Thank you, @zhengruifeng !

dongjoon-hyun added a commit that referenced this pull request Dec 11, 2025
### What changes were proposed in this pull request?

This PR aims to add `versionadded` to `TimeType`.

### Why are the changes needed?

For `Scala/Java` documentation, we have `since` information.

https://github.com/apache/spark/blob/22e26598785e11303409f43fc08ce627053dadb4/sql/api/src/main/scala/org/apache/spark/sql/types/TimeType.scala#L31-L34

We had better provide a correct information for the newly added `TimeType` in Python documentation.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #53436 from dongjoon-hyun/SPARK-52837.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
(cherry picked from commit d80a0f3)
Signed-off-by: Dongjoon Hyun <[email protected]>
@dongjoon-hyun
Copy link
Member Author

Merged to master/4.1 for Apache Spark 4.1.0.

@dongjoon-hyun dongjoon-hyun deleted the SPARK-52837 branch December 11, 2025 03:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants