This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-4.1
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-4.1 by this push:
new d738b2afc0b [fix](regression) Stabilize base compaction file size
limit case (#65386)
d738b2afc0b is described below
commit d738b2afc0b167ce7fb57e4281f2f176200b1dcc
Author: shuke <[email protected]>
AuthorDate: Mon Jul 13 14:01:37 2026 +0800
[fix](regression) Stabilize base compaction file size limit case (#65386)
## Summary
- Stabilize test_base_compaction_with_dup_key_max_file_size_limit by
explicitly setting BE compaction-related configs required by the case.
- Lower base_compaction_dup_key_max_file_size_mbytes to 512 during the
test so the loaded rowsets reliably hit the file-size filter.
- Restore original BE config values in finally to avoid polluting the
test environment.
## Testing
- ./run-regression-test.sh --run -d compaction -s
test_base_compaction_with_dup_key_max_file_size_limit
- Remote validation log:
/deploy/teamcity_agent13/work/a0477ef061788419/case_output/doris/output/regression-test/log/doris-regression-test.20260708.200842.log
- Result: Test 1 suites, failed 0 suites, fatal 0 scripts, skipped 0
scripts
---
...ase_compaction_with_dup_key_max_file_size_limit.groovy | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git
a/regression-test/suites/compaction/test_base_compaction_with_dup_key_max_file_size_limit.groovy
b/regression-test/suites/compaction/test_base_compaction_with_dup_key_max_file_size_limit.groovy
index a006ea406d6..24915e0df8d 100644
---
a/regression-test/suites/compaction/test_base_compaction_with_dup_key_max_file_size_limit.groovy
+++
b/regression-test/suites/compaction/test_base_compaction_with_dup_key_max_file_size_limit.groovy
@@ -19,6 +19,8 @@ import org.codehaus.groovy.runtime.IOGroovyMethods
suite("test_base_compaction_with_dup_key_max_file_size_limit", "p2") {
def tableName = "test_base_compaction_with_dup_key_max_file_size_limit"
+ def originalDisableAutoCompaction = null
+ def originalBaseCompactionFileSizeLimit = null
// use customer table of tpch_sf100
def rows = 15000000
@@ -71,6 +73,11 @@
suite("test_base_compaction_with_dup_key_max_file_size_limit", "p2") {
def backendId_to_backendHttpPort = [:]
getBackendIpHttpPort(backendId_to_backendIP,
backendId_to_backendHttpPort);
+ originalDisableAutoCompaction = get_be_param("disable_auto_compaction")
+ originalBaseCompactionFileSizeLimit =
get_be_param("base_compaction_dup_key_max_file_size_mbytes")
+ set_be_param("disable_auto_compaction", "true")
+ set_be_param("base_compaction_dup_key_max_file_size_mbytes", "512")
+
backend_id = backendId_to_backendIP.keySet()[0]
def (code, out, err) =
show_be_config(backendId_to_backendIP.get(backend_id),
backendId_to_backendHttpPort.get(backend_id))
@@ -177,7 +184,7 @@
suite("test_base_compaction_with_dup_key_max_file_size_limit", "p2") {
// cp: 5
trigger_and_wait_compaction(tableName, "cumulative")
- // Due to the limit of
config::base_compaction_dup_key_max_file_size_mbytes(1G),
+ // Due to the limit of
config::base_compaction_dup_key_max_file_size_mbytes,
// can not do base compaction, return E-808
// rowsets:
// [0-3] 2G nooverlapping
@@ -190,5 +197,11 @@
suite("test_base_compaction_with_dup_key_max_file_size_limit", "p2") {
def rowCount = sql "select count(*) from ${tableName}"
assertTrue(rowCount[0][0] != rows)
} finally {
+ if (originalBaseCompactionFileSizeLimit != null) {
+
set_original_be_param("base_compaction_dup_key_max_file_size_mbytes",
originalBaseCompactionFileSizeLimit)
+ }
+ if (originalDisableAutoCompaction != null) {
+ set_original_be_param("disable_auto_compaction",
originalDisableAutoCompaction)
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
