-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Issue #17049: ICU Date Cast #17067
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.
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
Issue #17049: ICU Date Cast #17067
Conversation
* Add missing DATE => TIMESTAMPTZ cast fixes: duckdb#17049 fixes: duckdblabs/duckdb-internal#4623
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good to me!
Just a clarifying question: From the tests I gather that there is no case where DATE -> TIMESTAMP_TZ can fail, is that correct?
@Mytherin can we rerun https://github.com/duckdb/duckdb/actions/runs/14388650645/job/40351540195?pr=17067
|
on duckdb-iceberg
|
Apparently not... D select date::timestamptz from all_types;
Invalid Input Error:
Type INT32 with value 5877642-06-25 (BC) can't be cast to the destination type INT64 I had convinced myself this was not an issue because none of the other casts worried about it but that's clearly wrong! |
* Add extreme date tests. fixes: duckdb#17049 fixes: duckdblabs/duckdb-internal#4623
@Tishj Changed the target branch and GitHub seems to be a bit grumpy about the lack of reviews... |
Thanks! |
Add bind_operator callback to TableFunction - allowing table functions to directly emit a LogicalOperator (duckdb/duckdb#17196) Issue duckdb/duckdb#17049: ICU Date Cast (duckdb/duckdb#17067) Notify JDBC repo to run Vendor.yml workflow (duckdb/duckdb#17099) Python.yml: Add back logic to perform fast-fail on Python 3.10 (duckdb/duckdb#17107)
Add bind_operator callback to TableFunction - allowing table functions to directly emit a LogicalOperator (duckdb/duckdb#17196) Issue duckdb/duckdb#17049: ICU Date Cast (duckdb/duckdb#17067) Notify JDBC repo to run Vendor.yml workflow (duckdb/duckdb#17099) Python.yml: Add back logic to perform fast-fail on Python 3.10 (duckdb/duckdb#17107)
Add bind_operator callback to TableFunction - allowing table functions to directly emit a LogicalOperator (duckdb/duckdb#17196) Issue duckdb/duckdb#17049: ICU Date Cast (duckdb/duckdb#17067) Notify JDBC repo to run Vendor.yml workflow (duckdb/duckdb#17099) Python.yml: Add back logic to perform fast-fail on Python 3.10 (duckdb/duckdb#17107)
fixes: #17049
fixes: duckdblabs/duckdb-internal#4623