You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "yiguolei (via GitHub)" <gi...@apache.org> on 2023/02/01 01:22:37 UTC
[GitHub] [doris] yiguolei commented on a diff in pull request #16289: [fuzzy](test) fuzzy some session variables stably according to pull_r…
yiguolei commented on code in PR #16289:
URL: https://github.com/apache/doris/pull/16289#discussion_r1092653124
##########
fe/fe-core/src/main/java/org/apache/doris/qe/SessionVariable.java:
##########
@@ -703,6 +703,14 @@ public void initFuzzyModeVariables() {
this.externalSortBytesThreshold = 100 * 1024 * 1024 * 1024;
break;
}
+ // pull_request_id default value is 0
Review Comment:
java 不需要这么写,pull_request_id 实际是static 类型,直接在initFuzzyModeVariables 里 Config.pull_request_id 就行
--
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-unsubscribe@doris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org