Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
morrySnow merged PR #32541: URL: https://github.com/apache/doris/pull/32541 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@doris.ap

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2024336511 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2022134314 TeamCity be ut coverage result: Function Coverage: 35.25% (8740/24797) Line Coverage: 27.03% (71540/264660) Region Coverage: 26.28% (37124/141278) Branch Coverage: 23.1

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2022122464 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit 8757f527612311c6bc80dbd0fd57cd3465c9a4ca with default session variables Stream

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2022112681 TPC-DS: Total hot run time: 181852 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-27 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2022098818 TPC-H: Total hot run time: 38451 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2022039726 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2021905754 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020217995 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit d6cf2fd693cc66303df92c3867707c5a87874428 with default session variables Stream

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020212899 TPC-DS: Total hot run time: 181636 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020191167 TPC-H: Total hot run time: 38376 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020170182 TeamCity be ut coverage result: Function Coverage: 35.27% (8738/24777) Line Coverage: 27.05% (71536/264504) Region Coverage: 26.28% (37115/141211) Branch Coverage: 23.1

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020154522 > add regression test please @924060929 I finish the test, review please. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020146348 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2020141845 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
924060929 commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2019876385 add regression test please -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific c

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2019866635 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-26 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2019866786 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016515048 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit a0540bd55cda57bac573d77902973f187ed9fb2d with default session variables Stream

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016514284 TPC-DS: Total hot run time: 186397 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016511370 TPC-H: Total hot run time: 37989 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016508279 TeamCity be ut coverage result: Function Coverage: 35.26% (8734/24767) Line Coverage: 27.06% (71524/264306) Region Coverage: 26.30% (37112/141100) Branch Coverage: 23.2

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016503804 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016502659 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016502795 TeamCity be ut coverage result: Function Coverage: 35.26% (8734/24767) Line Coverage: 27.06% (71515/264306) Region Coverage: 26.30% (37103/141100) Branch Coverage: 23.1

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016497385 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-23 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2016496086 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-22 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2014855223 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-22 Thread via GitHub
xiedeyantu commented on code in PR #32541: URL: https://github.com/apache/doris/pull/32541#discussion_r1535408640 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sum0.java: ## @@ -0,0 +1,125 @@ +// Licensed to the Apache Software Foundation (

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-22 Thread via GitHub
morrySnow commented on code in PR #32541: URL: https://github.com/apache/doris/pull/32541#discussion_r1535311423 ## fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Sum0.java: ## @@ -0,0 +1,125 @@ +// Licensed to the Apache Software Foundation (A

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2010879032 TeamCity be ut coverage result: Function Coverage: 35.27% (8729/24747) Line Coverage: 27.08% (71436/263767) Region Coverage: 26.33% (37076/140816) Branch Coverage: 23.2

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
xiedeyantu commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2010799393 run buildall -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To u

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2009091782 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
github-actions[bot] commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2009069879 clang-tidy review says "All clean, LGTM! :+1:" -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
doris-robot commented on PR #32541: URL: https://github.com/apache/doris/pull/32541#issuecomment-2009061495 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+process+y

[PR] [feature](agg) add aggregate function sum0 [doris]

2024-03-20 Thread via GitHub
xiedeyantu opened a new pull request, #32541: URL: https://github.com/apache/doris/pull/32541 ## Proposed changes Issue Number: close #xxx ## Further comments If this is a relatively large or complex change, kick off the discussion at [d...@doris.apache.org](mail