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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to