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.

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

fix: Support date_trunc #95

Merged
merged 5 commits into from
Aug 30, 2024
Merged

fix: Support date_trunc #95

merged 5 commits into from
Aug 30, 2024

Conversation

philippemnoel
Copy link
Collaborator

@philippemnoel philippemnoel commented Aug 23, 2024

Ticket(s) Closed

What

Makes date_trunc work

Why

See Github issue

How

Added Date to Timestamp and Timestamptz conversions.

Tests

See test_date_trunc

@philippemnoel philippemnoel changed the title fix: Fix/arrow date support fix: [HELP NEEDED] Fix/arrow date support Aug 23, 2024
@philippemnoel philippemnoel marked this pull request as draft August 23, 2024 19:20
@philippemnoel philippemnoel added help wanted Extra attention is needed good first issue Good for newcomers labels Aug 23, 2024
@philippemnoel
Copy link
Collaborator Author

Drafting this until someone picks it up

@rebasedming rebasedming self-assigned this Aug 30, 2024
@rebasedming rebasedming force-pushed the fix/arrow-date-support branch from 4af341b to af8cc8f Compare August 30, 2024 18:49
@rebasedming rebasedming changed the title fix: [HELP NEEDED] Fix/arrow date support fix: Support date_trunc Aug 30, 2024
@rebasedming rebasedming marked this pull request as ready for review August 30, 2024 18:51
@rebasedming rebasedming merged commit 8587ac7 into dev Aug 30, 2024
14 checks passed
@rebasedming rebasedming deleted the fix/arrow-date-support branch August 30, 2024 19:12
philippemnoel added a commit that referenced this pull request Sep 5, 2024
* update svg (#114)

* fix: Support `date_trunc` (#95)

* fixed date functions support

* fixing lint & more testing

* fix & remove unecessary conversion based on PR comments

* date trunc test

* remove unnecessary code

---------

Co-authored-by: Evance Soumaoro <evanxg852000@gmail.com>
Co-authored-by: Ming Ying <ming.ying.nyc@gmail.com>

* fix: Json & JsonB cast support (#33)

* fixed json & jsonb cast support

* fixing test

* fixing test

* adding a better test

* refactoring tests

* bug fixes

* json tests passing

* remove debug

* Increase the runner size

Signed-off-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>

---------

Signed-off-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>
Co-authored-by: Vipul Vaibhaw <vaibhaw.vipul@gmail.com>
Co-authored-by: Ming Ying <ming.ying.nyc@gmail.com>
Co-authored-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>

* ci: Rm cargo clean (#116)

* Rm cargo clean

* Remove unnecessary test

Signed-off-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>

---------

Signed-off-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>

* Prep v0.1.3 (#119)

---------

Signed-off-by: Philippe Noël <21990816+philippemnoel@users.noreply.github.com>
Co-authored-by: Ming <ming.ying.nyc@gmail.com>
Co-authored-by: Evance Soumaoro <evanxg852000@gmail.com>
Co-authored-by: Vipul Vaibhaw <vaibhaw.vipul@gmail.com>
philippemnoel added a commit that referenced this pull request Sep 5, 2024
* fixed date functions support

* fixing lint & more testing

* fix & remove unecessary conversion based on PR comments

* date trunc test

* remove unnecessary code

---------

Co-authored-by: Evance Soumaoro <evanxg852000@gmail.com>
Co-authored-by: Ming Ying <ming.ying.nyc@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DATE_TRUNC('day', date_day) not supported
3 participants