[bugfix](VecDateTimeValue) eat the value of microsecond in function from_date_format_str #13504
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Issue Number: none
Problem summary
case: select str_to_date("2022-05-24 10:00:00.123 PM", "%Y-%m-%d %h:%i:%s.%f %p");
disable_vectorized
Expected: 2022-05-24 22:00:00.123000
Actual: 2022-05-24 22:00:00.123000
enables_vectorized
Expected: 2022-05-24 22:00:00
Actual: null
In function VecDateTimeValue::from_date_format_str, it is necessary to eat the microsecond "123" before parsing "PM", even though _microsecond has been removed in class VecDateTimeValue.
Checklist(Required)