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
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
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()) {
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
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
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
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
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
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
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
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
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
12 matches
Mail list logo