Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

sql: consider adding a try_parse_mysql_datetime function #27026

Open
chuck-alt-delete opened this issue May 10, 2024 · 1 comment
Open

sql: consider adding a try_parse_mysql_datetime function #27026

chuck-alt-delete opened this issue May 10, 2024 · 1 comment
Labels
C-feature Category: new feature or request

Comments

@chuck-alt-delete
Copy link
Contributor

chuck-alt-delete commented May 10, 2024 •
edited by benesch

Feature request

The string produced with TEXT COLUMNS (a_timestamp_column) does not match the format try_parse requires to enable temporal filter pushdown.

We should consider adding a new try_parse_mysql_datetime function that handles the MySQL datetime format.
@chuck-alt-delete chuck-alt-delete added the C-feature Category: new feature or request label May 10, 2024
@chuck-alt-delete chuck-alt-delete mentioned this issue May 10, 2024
Open
@benesch
Copy link
Member

benesch commented May 10, 2024

As @rjobanp mentioned on Slack, the TEXT COLUMNS option is performing to spec. The intent is to exactly match the MySQL textual format for the column.

Going to adjust this ask to be about adding something more akin to a try_parse_mysql_datetime function!

@benesch benesch changed the title MySQL timestamp text column: parse timestamp in iso 8601 format sql: consider adding a try_parse_mysql_datetime function May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-feature Category: new feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@benesch @chuck-alt-delete