Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-15 Thread via GitHub
yiguolei merged PR #25306: URL: https://github.com/apache/doris/pull/25306 -- 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.apa

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-15 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1763653056 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-15 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1763653025 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761309347 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 46.85 seconds stream load tsv: 554 seconds loaded 74807831229 Bytes,

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761303328 TeamCity be ut coverage result: Function Coverage: 36.29% (8146/22450) Line Coverage: 28.51% (65310/229093) Region Coverage: 27.20% (33832/124404) Branch Coverage: 23.9

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761281806 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
zhangstar333 commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761271407 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

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761224669 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.78 seconds stream load tsv: 553 seconds loaded 74807831229 Bytes,

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761132910 TeamCity be ut coverage result: Function Coverage: 36.29% (8144/22439) Line Coverage: 28.50% (65277/229061) Region Coverage: 27.19% (33816/124366) Branch Coverage: 23.9

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761115740 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-13 Thread via GitHub
zhangstar333 commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1761096381 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

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
HappenLee commented on code in PR #25306: URL: https://github.com/apache/doris/pull/25306#discussion_r1357829182 ## be/src/vec/functions/function_quantile_state.cpp: ## @@ -191,14 +191,28 @@ class FunctionQuantileStatePercent : public IFunction { } auto str_col

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1760737420 TeamCity be ut coverage result: Function Coverage: 36.29% (8145/22446) Line Coverage: 28.42% (65253/229603) Region Coverage: 27.10% (33798/124734) Branch Coverage: 23.8

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1760716417 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.28 seconds stream load tsv: 558 seconds loaded 74807831229 Bytes,

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1760698088 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
zhangstar333 commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1760693513 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

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1759338473 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.25 seconds stream load tsv: 555 seconds loaded 74807831229 Bytes,

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1759319008 TeamCity be ut coverage result: Function Coverage: 36.23% (8156/22510) Line Coverage: 28.33% (65304/230516) Region Coverage: 27.04% (33815/125048) Branch Coverage: 23.8

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
github-actions[bot] commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1759231708 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] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-12 Thread via GitHub
zhangstar333 commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1759219455 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

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-11 Thread via GitHub
doris-robot commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1757472204 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 45.98 seconds stream load tsv: 558 seconds loaded 74807831229 Bytes,

Re: [PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-11 Thread via GitHub
zhangstar333 commented on PR #25306: URL: https://github.com/apache/doris/pull/25306#issuecomment-1757410007 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

[PR] [fix](functions) add quantile_state_empty function signature [doris]

2023-10-11 Thread via GitHub
zhangstar333 opened a new pull request, #25306: URL: https://github.com/apache/doris/pull/25306 ## Proposed changes quantile_state_empty = percentile_empty to_quantile_state= percentile_hash quantile_percent= percentile_approx_raw Issue Number: close #xxx