morrySnow commented on code in PR #49415:
URL: https://github.com/apache/doris/pull/49415#discussion_r2013594845
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/stats/StatsCalculator.java:
##########
@@ -301,13 +301,32 @@ public static void estimate(GroupExpression
groupExpression, CascadesContext con
private void estimate() {
Plan plan = groupExpression.getPlan();
- Statistics newStats = plan.accept(this, null);
+ Statistics newStats;
+ try {
+ newStats = plan.accept(this, null);
+ } catch (Exception e) {
+ // throw exception in debug mode
Review Comment:
log warn here
##########
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:
use as static var as name, and let name be lowercase
--
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]
