Uh oh!
There was an error while loading. Please reload this page.
[fix](timezone) fix timezone parse when there is no tzfile - #24578
Conversation
zclllyybb
commented
Sep 18, 2023
run buildall |
zclllyybb
commented
Sep 18, 2023
@jacktengg PTAL |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Sep 18, 2023
TeamCity be ut coverage result: |
doris-robot
commented
Sep 18, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
PR approved by anyone and no changes requested. |
HappenLee
left a comment
There was a problem hiding this comment.
ADD a UT or regression test
zclllyybb
commented
Sep 19, 2023
run buildall |
PR approved by at least one committer and no changes requested. |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Sep 19, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
zclllyybb
commented
Sep 19, 2023
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Sep 19, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
zclllyybb
commented
Sep 19, 2023
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
doris-robot
commented
Sep 19, 2023
TeamCity be ut coverage result: |
doris-robot
commented
Sep 19, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
zclllyybb
commented
Sep 20, 2023
run buildall |
doris-robot
commented
Sep 20, 2023
TeamCity be ut coverage result: |
doris-robot
commented
Sep 20, 2023
(From new machine)TeamCity pipeline, clickbench performance test result: |
Proposed changes
Issue Number: close #xxx
When there's no tzfile:
before:
after:
It tested in docker.
Further comments
If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...