-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[Fix-3.0](function) Fix wrong calculation of timestampdiff of datetimev1 #54967
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
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 40617 ms |
TPC-DS: Total hot run time: 192186 ms |
ClickBench: Total hot run time: 29.68 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 39988 ms |
TPC-DS: Total hot run time: 192579 ms |
ClickBench: Total hot run time: 29.58 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
|
run buildall |
TPC-H: Total hot run time: 40122 ms |
TPC-DS: Total hot run time: 192821 ms |
ClickBench: Total hot run time: 29.72 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run buildall |
TPC-H: Total hot run time: 40099 ms |
|
run buildall |
TPC-H: Total hot run time: 40341 ms |
TPC-DS: Total hot run time: 190230 ms |
ClickBench: Total hot run time: 29.02 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
FE UT Coverage ReportIncrement line coverage |
|
run buildall |
TPC-H: Total hot run time: 40106 ms |
TPC-DS: Total hot run time: 190724 ms |
ClickBench: Total hot run time: 30.04 s |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
only fix in 3.1 by #56893 |
…ev1 (apache#5237) pick apache#54967 Related PR: apache#49429 Problem Summary: before: ```sql mysql> select hours_diff(now(), date_format(event_date, '%Y-%m-%d %H:00:00')) from my_doris_table; +-----------------------------------------------------------------+ | hours_diff(now(), date_format(event_date, '%Y-%m-%d %H:00:00')) | +-----------------------------------------------------------------+ | 103079215078 | +-----------------------------------------------------------------+ ``` now: ```sql mysql> select hours_diff(now(), date_format(event_date, '%Y-%m-%d %H:00:00')) from my_doris_table; +-----------------------------------------------------------------+ | hours_diff(now(), date_format(event_date, '%Y-%m-%d %H:00:00')) | +-----------------------------------------------------------------+ | -24 | +-----------------------------------------------------------------+ ``` and make fe fold of `now()` return v2 type ### Release note Fix wrong calculation of timestampdiff of datetimev1
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #49429
Problem Summary:
before:
now:
and make fe fold of
now()return v2 typeRelease note
Fix wrong calculation of timestampdiff of datetimev1
Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)