-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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 #561: ICU Date addition #2730
Conversation
Clean up the templates to have better names and type inference.
Implement TIMESTAMPTZ + INTERVAL using ICU.
Backout Window extension break.
…to hawkfish-icu-datediff
Implement TIMESTAMPTZ - INTERVAL Test commutative, subtraction and pre-epoch addition.
… into hawkfish-time-zone
…ckdb into hawkfish-icu-dateadd
I don't understand why the coverage went down - none of the files in the PR are listed! |
The project coverage is a bit finnicky because there are race conditions in the coverage that cause minor fluctuations (i.e. sometimes certain lines of code will not be executed because of threading). I have tried before to prevent it from pushing a failure in case of a minor decrease for that reason, but could not manage to get codecov to obey as of yet. |
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 for the PR! Looks great. One request for more tests:
Add limits tests.
Fix Windows linkage.
Thanks! |
Implement adding and subtracting
INTERVAL
to and fromTIMESTAMPTZ