sollhui opened a new pull request, #61814: URL: https://github.com/apache/doris/pull/61814
## Problem `enable_strict_consistency_dml` is designed to handle multi-replica consistency for DML operations. In cloud mode (store-compute separation), data has only a single copy, so there is no multi-replica consistency concern. Keeping this variable `true` in cloud mode causes unnecessary data shuffling overhead. ## Solution Override `isEnableStrictConsistencyDml()` to return `false` when running in cloud mode (`Config.isCloudMode()`), regardless of the session variable's value. This follows the same pattern as `isDisableFileCache()`. Update all call sites in `RequestPropertyDeriver` to use the getter instead of direct field access so the cloud-mode check takes effect. ## Tests Added two unit tests in `SessionVariablesTest`: - Verify `isEnableStrictConsistencyDml()` returns `false` in cloud mode (even when field is `true`) - Verify `isEnableStrictConsistencyDml()` follows the field value in non-cloud mode -- 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]
