8000 Issue #17049: ICU Date Cast by hawkfish · Pull Request #17067 · duckdb/duckdb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

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

Merged
merged 3 commits into from
Apr 21, 2025
Merged

Issue #17049: ICU Date Cast #17067

merged 3 commits into from
Apr 21, 2025

Conversation

hawkfish
Copy link
Contributor
  • Add missing DATE => TIMESTAMPTZ cast

fixes: #17049
fixes: duckdblabs/duckdb-internal#4623

* Add missing DATE => TIMESTAMPTZ cast

fixes: duckdb#17049
fixes: duckdblabs/duckdb-internal#4623
Copy link
Contributor
@Tishj Tishj left a 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?

@Tishj
Copy link
Contributor
Tishj commented Apr 15, 2025

@Mytherin can we rerun https://github.com/duckdb/duckdb/actions/runs/14388650645/job/40351540195?pr=17067
looking at the logs, an http issue occurred:

error: https://github.com/aws/s2n-tls/archive/v1.5.5.tar.gz: curl failed to download with exit code 52
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

curl: (52) Empty reply from server

@Tishj
Copy link
Contributor
Tishj commented Apr 15, 2025

I have seen this failure before https://productionresultssa5.blob.core.windows.net/action 8000 s-results/59ce9602-9378-4a5c-af42-bcd10619a579/workflow-job-run-61b6088a-0ade-5cb3-591b-f12fe41a14d9/logs/job/job-logs.txt?rsct=text%2Fplain&se=2025-04-15T10%3A10%3A30Z&sig=NXbN497xlwDYVxSYmTeESCVLPwoFQ4Kj8yU4H%2FlznDQ%3D&ske=2025-04-15T21%3A09%3A28Z&skoid=ca7593d4-ee42-46cd-af88-8b886a2f84eb&sks=b&skt=2025-04-15T09%3A09%3A28Z&sktid=398a6654-997b-47e9-b12b-9515b896b4de&skv=2025-01-05&sp=r&spr=https&sr=b&st=2025-04-15T10%3A00%3A25Z&sv=2025-01-05

on duckdb-iceberg

2025-04-10T22:14:56.9594670Z --------------------
2025-04-10T22:14:56.9594964Z STDERR
2025-04-10T22:14:56.9595279Z --------------------
2025-04-10T22:14:56.9595793Z ================================================================================
2025-04-10T22:14:56.9596131Z 
2025-04-10T22:14:56.9597042Z Query unexpectedly failed! (D:/a/duckdb/duckdb/build/release/_deps/aws_extension_fc-src/test/sql/aws_secret_aws.test:11)!
2025-04-10T22:14:56.9597824Z 
2025-04-10T22:14:56.9598201Z ================================================================================
2025-04-10T22:14:56.9598560Z 
2025-04-10T22:14:56.9598742Z CREATE SECRET s1 (
2025-04-10T22:14:56.9598937Z 
2025-04-10T22:14:56.9599085Z     TYPE AWS,
2025-04-10T22:14:56.9599252Z 
2025-04-10T22:14:56.9599476Z     PROVIDER credential_chain
2025-04-10T22:14:56.9599728Z 
2025-04-10T22:14:56.9599847Z );
2025-04-10T22:14:56.9599993Z 
2025-04-10T22:14:56.9600333Z ================================================================================
2025-04-10T22:14:56.9600675Z 
2025-04-10T22:14:56.9600972Z Invalid Input Error: Secret type 'aws' not found

@hawkfish
Copy link
Contributor Author

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?

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!

@duckdb-draftbot duckdb-draftbot marked this pull request as draft April 16, 2025 23:59
@hawkfish hawkfish marked this pull request as ready for review April 17, 2025 00:00
@hawkfish hawkfish changed the base branch from v1.2-histrionicus to main April 17, 2025 22:26
@hawkfish hawkfish requested a review from Tishj April 17, 2025 22:31
@hawkfish
Copy link
Contributor Author

@Tishj Changed the target branch and GitHub seems to be a bit grumpy about the lack of reviews...

@Mytherin Mytherin merged commit 1a9dc44 into duckdb:main Apr 21, 2025
2 checks passed
@Mytherin
Copy link
Collaborator

Thanks!

@hawkfish hawkfish deleted the date-timestamptz branch April 22, 2025 01:35
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 18, 2025
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)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 18, 2025
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)
krlmlr added a commit to duckdb/duckdb-r that referenced this pull request May 19, 2025
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)
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.

PostgreSQL incompatibilities in DATE_TRUNC and DATE::TIMESTAMPTZ
3 participants
0