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

GH-37608: [C++][Gandiva] TO_DATE function supports YYYY-MM and YYYY (… #51

Closed
wants to merge 1 commit into from
Closed

GH-37608: [C++][Gandiva] TO_DATE function supports YYYY-MM and YYYY (… #51

wants to merge 1 commit into from

Commits on Sep 26, 2023

  1. apacheGH-37608: [C++][Gandiva] TO_DATE function supports YYYY-MM and … 

    …YYYY (apache#37609)

Added fix for case when pattern does not contain day part

### Rationale for this change

TO_DATE Gandiva function returns wrong result if used with pattern 'YYYY-MM' or 'YYYY'.

### What changes are included in this PR?

Add a fix for case when tm_mday is zero to set it to 1

### Are these changes tested?

Added tests that cover described cases

### Are there any user-facing changes?

No

* Closes: apache#37608

Authored-by: DenisTarasyuk <131180287+DenisTarasyuk@users.noreply.github.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
    @DenisTarasyuk
    DenisTarasyuk committed Sep 26, 2023
    Configuration menu
    Copy the full SHA
    78a2495 View commit details
    Browse the repository at this point in the history