wangbo commented on code in PR #44028: URL: https://github.com/apache/doris/pull/44028#discussion_r1846026881
########## gensrc/thrift/BackendService.thrift: ########## @@ -265,8 +265,8 @@ struct TWorkloadGroupInfo { 9: optional i32 scan_thread_num 10: optional i32 max_remote_scan_thread_num 11: optional i32 min_remote_scan_thread_num - 12: optional i32 spill_threshold_low_watermark - 13: optional i32 spill_threshold_high_watermark + 12: optional i32 memory_low_watermark + 13: optional i32 memory_high_watermark Review Comment: Did a upgrade test here? Such as a old FE and a new BE with this PR, what could happen? -- 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