Skip to content

Conversation

@zhiqiang-hhhh
Copy link
Contributor

@zhiqiang-hhhh zhiqiang-hhhh commented Mar 6, 2024

before
image

now they work as mysql

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

@zhiqiang-hhhh
Copy link
Contributor Author

run buildall

Copy link
Contributor

@github-actions github-actions bot left a 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

@wm1581066 wm1581066 added dev/2.0.x usercase Important user case type label labels Mar 6, 2024
@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 35.41% (8559/24174)
Line Coverage: 27.16% (69490/255812)
Region Coverage: 26.39% (36054/136626)
Branch Coverage: 23.23% (18425/79332)
Coverage Report: http://coverage.selectdb-in.cc/coverage/732b05b3397faa49676e20880338a5793be18b42_732b05b3397faa49676e20880338a5793be18b42/report/index.html

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2024

PR approved by anyone and no changes requested.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 7, 2024

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Mar 7, 2024
@yiguolei yiguolei merged commit b4ce532 into apache:master Mar 7, 2024
zhiqiang-hhhh added a commit to zhiqiang-hhhh/doris that referenced this pull request Mar 7, 2024
@zhiqiang-hhhh zhiqiang-hhhh deleted the money-format branch March 7, 2024 04:02
yiguolei pushed a commit that referenced this pull request Mar 7, 2024
yiguolei pushed a commit that referenced this pull request Mar 7, 2024
yiguolei pushed a commit that referenced this pull request Mar 7, 2024
@yiguolei yiguolei mentioned this pull request Mar 24, 2024
// truncate to 2 decimal places, keep same with mysql
double value =
MathFunctions::my_double_round(data_column->get_element(i), 2, false, false);
MathFunctions::my_double_round(data_column->get_element(i), 2, false, true);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should do round instead of truncate

mongo360 pushed a commit to mongo360/doris that referenced this pull request Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/2.0.6-merged reviewed usercase Important user case type label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants