Skip to content

Conversation

@Omega359
Copy link
Contributor

@Omega359 Omega359 commented Dec 6, 2025

Which issue does this PR close?

Rationale for this change

Improved type support for to_date function.

What changes are included in this PR?

Code, slt, updated docs.

Are these changes tested?

Yes

Are there any user-facing changes?

More types supported.

@github-actions github-actions bot added documentation Improvements or additions to documentation sqllogictest SQL Logic Tests (.slt) functions Changes to functions implementation labels Dec 6, 2025
@Omega359 Omega359 marked this pull request as ready for review December 6, 2025 17:12
Copy link
Contributor

@Jefffrey Jefffrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor suggestion

}
}
}
Float16 | Float32 | Float64 => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Float16 | Float32 | Float64 => {
Float16 | Float32 | Float64 | Decimal32(_, _) | Decimal64(_, _) | Decimal128(_, _) | Decimal256(_, _) => {

Then in the description can simplify as:

Supports strings, numeric and timestamp types as input.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation functions Changes to functions implementation sqllogictest SQL Logic Tests (.slt)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update to_date udf function to support a consistent set of argument types

2 participants