HappenLee commented on code in PR #66788:
URL: https://github.com/apache/doris/pull/66788#discussion_r3790986175
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -3513,6 +3515,10 @@ public void
checkAnnIndexCandidateRowsPercentThreshold(String value) {
description = "Enable extended regular expressions, support
look-around zero-width assertions")
public boolean enableExtendedRegex = false;
+ @VarAttrDef.VarAttr(name = ENABLE_HYPERSCAN_FALLBACK, needForward = true,
affectQueryResultInExecution = true,
Review Comment:
Thanks for identifying this. This exposes a broader MTMV session-variable
persistence boundary: MTMV currently captures and restores only variables
marked `affectQueryResultInPlan`, while `enable_hyperscan_fallback` is
correctly classified as execution-affecting. We should discuss separately which
execution-affecting variables an asynchronous MTMV refresh must inherit, rather
than reclassifying this option as plan-affecting or adding an isolated
workaround in this PR. We will track and handle that broader policy in a
follow-up discussion/change, so resolving this thread for the current PR.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
