morningman commented on code in PR #32010:
URL: https://github.com/apache/doris/pull/32010#discussion_r1549365608
##########
fe/fe-common/src/main/java/org/apache/doris/common/Config.java:
##########
@@ -2531,6 +2531,7 @@ public class Config extends ConfigBase {
"Whether to enable proxy protocol"
})
public static boolean enable_proxy_protocol = false;
+ public static int profile_async_collect_expire_time_secs = 5;
Review Comment:
Missing `@ConfField`?
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -634,6 +635,10 @@ public class SessionVariable implements Serializable,
Writable {
@VariableMgr.VarAttr(name = ENABLE_PROFILE, needForward = true)
public boolean enableProfile = false;
+ // if true, need report to coordinator when plan fragment execute
successfully.
Review Comment:
```suggestion
// if > 0, need report to coordinator when plan fragment execute
successfully.
```
--
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]
