Copilot commented on code in PR #50113:
URL: https://github.com/apache/doris/pull/50113#discussion_r2059839183
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -2245,6 +2247,9 @@ public boolean isEnableHboNonStrictMatchingMode() {
"When processing both \\n and \\r\\n as CSV line
separators, should \\r be retained?"})
public boolean keepCarriageReturn = false;
+ @VariableMgr.VarAttr(name = EXCHANGE_MULTI_BLOCKS_BYTE_SIZE,
+ description = {"enable exchange send multi blocks in one rpc,
default is 256KB. negatives means disable"})
Review Comment:
[nitpick] Consider revising the description text to improve clarity, e.g.,
use "a negative value disables multi-block exchange" instead of "negatives
means disable".
```suggestion
description = {"Enable exchange to send multiple blocks in one
RPC. Default is 256KB. A negative value disables multi-block exchange."})
```
--
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]
