Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-05-31 Thread via GitHub
Mryange closed pull request #33557: [feature](expr) Add length limit to the debug string of expr URL: https://github.com/apache/doris/pull/33557 -- 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 s

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
Mryange commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2062808703 run buidlall -- 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2061714922 TeamCity be ut coverage result: Function Coverage: 35.44% (8905/25130) Line Coverage: 27.12% (73125/269621) Region Coverage: 26.25% (37803/144002) Branch Coverage: 23.0

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2061667341 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit 83c7e0f6928d11071a9e52b9375c662535cd86dd with default session variables Stream

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2061658628 ClickBench: Total hot run time: 29.93 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
Mryange commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2061491539 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2060823011 Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Load test result on commit 83c7e0f6928d11071a9e52b9375c662535cd86dd with default session variables Stream

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2060817824 TeamCity be ut coverage result: Function Coverage: 35.45% (8907/25128) Line Coverage: 27.14% (73149/269552) Region Coverage: 26.27% (37827/143972) Branch Coverage: 23.0

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2060817176 ClickBench: Total hot run time: 30.46 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
Mryange commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2060646207 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-17 Thread via GitHub
github-actions[bot] commented on code in PR #33557: URL: https://github.com/apache/doris/pull/33557#discussion_r1568390651 ## be/src/vec/exprs/vcolumn_ref.h: ## @@ -16,6 +16,8 @@ // under the License. #pragma once +#include Review Comment: warning: 'fmt/format.h' file n

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-11 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2049935414 TeamCity be ut coverage result: Function Coverage: 35.59% (8904/25016) Line Coverage: 27.31% (73107/267706) Region Coverage: 26.44% (37814/143029) Branch Coverage: 23.1

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-11 Thread via GitHub
github-actions[bot] commented on code in PR #33557: URL: https://github.com/apache/doris/pull/33557#discussion_r1561064336 ## be/src/vec/exprs/vcast_expr.cpp: ## @@ -129,20 +129,21 @@ const std::string& VCastExpr::expr_name() const { return _expr_name; } -std::string VCa

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-11 Thread via GitHub
Mryange commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2049744342 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 unsu

[PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-11 Thread via GitHub
Mryange opened a new pull request, #33557: URL: https://github.com/apache/doris/pull/33557 ## Proposed changes The debug string may be too long, so we'll truncate it here. ## Further comments If this is a relatively large or complex change, kick off the discussion

Re: [PR] [feature](expr) Add length limit to the debug string of expr [doris]

2024-04-11 Thread via GitHub
doris-robot commented on PR #33557: URL: https://github.com/apache/doris/pull/33557#issuecomment-2049735003 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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-04-09 Thread via GitHub
Mryange closed pull request #30464: [feature](expr) Add length limit to the debug string of expr. URL: https://github.com/apache/doris/pull/30464 -- 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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-02-20 Thread via GitHub
doris-robot commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-1955851383 TeamCity be ut coverage result: Function Coverage: 35.71% (8557/23963) Line Coverage: 27.59% (69409/251588) Region Coverage: 26.72% (36023/134792) Branch Coverage: 23.5

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-02-20 Thread via GitHub
Mryange commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-1955759453 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
doris-robot commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-191315 ClickBench: Total hot run time: 30.67 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
doris-robot commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-1913151498 TeamCity be ut coverage result: Function Coverage: 36.18% (8621/23827) Line Coverage: 28.26% (70515/249565) Region Coverage: 27.24% (36383/133544) Branch Coverage: 24.0

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
doris-robot commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1913143965 TeamCity be ut coverage result: Function Coverage: 36.18% (8621/23827) Line Coverage: 28.26% (70516/249565) Region Coverage: 27.25% (36386/133544) Branch Coverage: 24.0

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
github-actions[bot] commented on code in PR #30464: URL: https://github.com/apache/doris/pull/30464#discussion_r1468448002 ## be/src/vec/exprs/vcase_expr.cpp: ## @@ -125,20 +125,21 @@ const std::string& VCaseExpr::expr_name() const { return _expr_name; } -std::string VCa

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
Mryange commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-1913128490 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
Mryange commented on PR #30464: URL: https://github.com/apache/doris/pull/30464#issuecomment-1913127328 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 unsu

[PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
Mryange opened a new pull request, #30464: URL: https://github.com/apache/doris/pull/30464 ## 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](mailto:

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
Mryange closed pull request #28395: [feature](expr) Add length limit to the debug string of expr. URL: https://github.com/apache/doris/pull/28395 -- 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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2024-01-27 Thread via GitHub
Mryange commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1913121412 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-15 Thread via GitHub
doris-robot commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1858032703 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 0cc420fbc2747d0ae5c286e0cbaefef4e6b839cd, data reload: false

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-15 Thread via GitHub
Mryange commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1857896568 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-14 Thread via GitHub
Mryange commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1857341703 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-14 Thread via GitHub
github-actions[bot] commented on code in PR #28395: URL: https://github.com/apache/doris/pull/28395#discussion_r1427522692 ## be/src/vec/exprs/vcase_expr.cpp: ## @@ -117,20 +117,21 @@ const std::string& VCaseExpr::expr_name() const { return _expr_name; } -std::string VCa

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-14 Thread via GitHub
Mryange commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1857228468 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 unsu

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-14 Thread via GitHub
yiguolei commented on code in PR #28395: URL: https://github.com/apache/doris/pull/28395#discussion_r1426922885 ## be/src/vec/exprs/vcast_expr.cpp: ## @@ -142,8 +142,11 @@ std::string VCastExpr::debug_string() const { out << ","; } out << input_exp

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

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

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-14 Thread via GitHub
doris-robot commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1855443560 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 7d25f17d7d23589b7cfbfc324855a5e4ca566914, data reload: false

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-13 Thread via GitHub
github-actions[bot] commented on code in PR #28395: URL: https://github.com/apache/doris/pull/28395#discussion_r1426268241 ## be/src/vec/exprs/vexpr.cpp: ## @@ -444,19 +444,31 @@ std::string VExpr::debug_string() const { out << " children=" << debug_string(_children);

Re: [PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-13 Thread via GitHub
Mryange commented on PR #28395: URL: https://github.com/apache/doris/pull/28395#issuecomment-1855262685 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 unsu

[PR] [feature](expr) Add length limit to the debug string of expr. [doris]

2023-12-13 Thread via GitHub
Mryange opened a new pull request, #28395: URL: https://github.com/apache/doris/pull/28395 ## 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](mailto: