Re: [PR] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-31 Thread via GitHub
Mryange closed pull request #45334: [fix](expr) Fix the problem of concurrent execution in VExpr. URL: https://github.com/apache/doris/pull/45334 -- 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] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-17 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2548139486 After some discussion, this PR will be split into two PRs. The first PR will remove the _can_fast_execute variable. The second PR will add const to exec. -- This is an automate

Re: [PR] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-16 Thread via GitHub
airborne12 commented on code in PR #45334: URL: https://github.com/apache/doris/pull/45334#discussion_r1887850501 ## be/src/vec/exprs/vcompound_pred.h: ## @@ -148,14 +149,14 @@ class VCompoundPred : public VectorizedFnCall { if (all_pass && !res.is_empty()) {

Re: [PR] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-12 Thread via GitHub
github-actions[bot] commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2540385038 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](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-12 Thread via GitHub
github-actions[bot] commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2540385002 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](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-12 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2538553165 run cloud_p0 -- 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] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-12 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2538552924 run p0 -- 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 unsubscrib

Re: [PR] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-11 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2538044374 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] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-11 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2537948230 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] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-11 Thread via GitHub
github-actions[bot] commented on code in PR #45334: URL: https://github.com/apache/doris/pull/45334#discussion_r1881352557 ## be/src/vec/exprs/vcompound_pred.h: ## @@ -57,7 +57,8 @@ class VCompoundPred : public VectorizedFnCall { const std::string& expr_name() const overr

Re: [PR] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-11 Thread via GitHub
Mryange commented on PR #45334: URL: https://github.com/apache/doris/pull/45334#issuecomment-2537763819 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] [fix](expr) Fix the problem of concurrent execution in VExpr. [doris]

2024-12-11 Thread via GitHub
Mryange opened a new pull request, #45334: URL: https://github.com/apache/doris/pull/45334 ### What problem does this PR solve? In the past, a _can_fast_execute flag was maintained in VExpr. However, since exec executes concurrently, errors would occur when using the _can_fast_exe