morrySnow commented on code in PR #49415:
URL: https://github.com/apache/doris/pull/49415#discussion_r2009924085
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -2271,6 +2271,11 @@ public void setDetailShapePlanNodes(String
detailShapePlanNodes) {
"use other health replica when the use_fix_replica meet error"
})
public boolean fallbackOtherReplicaWhenFixedCorrupt = false;
+ @VariableMgr.VarAttr(name = "FE_DEBUG", needForward = true, fuzzy = true,
Review Comment:
```suggestion
public static final String DEBUG_NEREIDS_THROW_ALL_EXCEPTIONS =
"debug_nereids_throw_all_exceptions";
@VariableMgr.VarAttr(name = DEBUG_NEREIDS_THROW_ALL_EXCEPTIONS,
needForward = true, fuzzy = true,
```
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -2271,6 +2271,11 @@ public void setDetailShapePlanNodes(String
detailShapePlanNodes) {
"use other health replica when the use_fix_replica meet error"
})
public boolean fallbackOtherReplicaWhenFixedCorrupt = false;
+ @VariableMgr.VarAttr(name = "FE_DEBUG", needForward = true, fuzzy = true,
+ description = {"when set true, FE will throw exceptions instead
swallow them. This is used for test",
+ "when set true, FE will throw exceptions instead swallow
them. This is used for test"})
Review Comment:
description should one english and one chinese
--
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]
