-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](function) fix date_format and from_unixtime core when meet long format string #35883
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. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
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.
clang-tidy made some suggestions
TPC-H: Total hot run time: 41096 ms |
TPC-DS: Total hot run time: 168908 ms |
ClickBench: Total hot run time: 31.59 s |
|
run buildall |
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.
clang-tidy made some suggestions
TPC-H: Total hot run time: 41794 ms |
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 173175 ms |
ClickBench: Total hot run time: 30.45 s |
zhiqiang-hhhh
left a comment
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.
LGTM
|
PR approved by anyone and no changes requested. |
|
run buildall |
|
TeamCity be ut coverage result: |
TPC-H: Total hot run time: 41385 ms |
TPC-DS: Total hot run time: 171704 ms |
ClickBench: Total hot run time: 30.53 s |
|
run buildall |
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.
clang-tidy made some suggestions
TPC-H: Total hot run time: 41025 ms |
TPC-DS: Total hot run time: 171259 ms |
|
TeamCity be ut coverage result: |
ClickBench: Total hot run time: 30.84 s |
|
run buildall |
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.
clang-tidy made some suggestions
TPC-H: Total hot run time: 40538 ms |
|
TeamCity be ut coverage result: |
TPC-DS: Total hot run time: 173881 ms |
ClickBench: Total hot run time: 30.5 s |
HappenLee
left a comment
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.
LGTM
|
PR approved by at least one committer and no changes requested. |
… format string (apache#35883) ## Proposed changes before: crash now: ```sql mysql> SELECT DATE_FORMAT('2011-06-24', '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b'); +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | date_format(cast('2011-06-24' as DATETIMEV2(0)), '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b') | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | NULL | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ 1 row in set (0.80 sec) ```
… format string (#35883) ## Proposed changes before: crash now: ```sql mysql> SELECT DATE_FORMAT('2011-06-24', '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b'); +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | date_format(cast('2011-06-24' as DATETIMEV2(0)), '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b') | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | NULL | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ 1 row in set (0.80 sec) ```
… format string (apache#35883) before: crash now: ```sql mysql> SELECT DATE_FORMAT('2011-06-24', '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b'); +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | date_format(cast('2011-06-24' as DATETIMEV2(0)), '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b') | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | NULL | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ 1 row in set (0.80 sec) ```
… format string (apache#35883) before: crash now: ```sql mysql> SELECT DATE_FORMAT('2011-06-24', '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b'); +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | date_format(cast('2011-06-24' as DATETIMEV2(0)), '%f %V %f %l %V %I %S %p %w %r %j %f %l %I %D %w %j %D %e %s %V %f %D %M %s %X %U %v %c %u %x %r %j %a %h %s %m %a %v %u %b') | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ | NULL | +--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+ 1 row in set (0.80 sec) ```
…et long format string (apache#35883) (apache#37178) pick apache#35883
Proposed changes
Issue Number: close #xxx
before:
crash
now: