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 14bc641961d [fix](case) Relax show data compaction size check (#65387)
14bc641961d is described below
commit 14bc641961d84ccfcdc236d92353e2cc9ae238a5
Author: shuke <[email protected]>
AuthorDate: Fri Jul 10 10:29:00 2026 +0800
[fix](case) Relax show data compaction size check (#65387)
## Summary
- Relax strict SHOW DATA size equality in
test_show_data_with_compaction.
- Assert the compacted documents table size falls into the observed
stable physical range for cloud/non-cloud runs.
- Keep the small insert table comparison as a relative size check after
full compaction.
## Testing
- [x] git diff --check --
regression-test/suites/inverted_index_p2/test_show_data.groovy
- [x] Remote branch-4.1 regression single suite before final range-only
adjustment: ./run-regression-test.sh --run -d inverted_index_p2 -s
test_show_data_with_compaction -parallel 1
- Result: All suites success. Test 1 suites, failed 0 suites, fatal 0
scripts, skipped 0 scripts
---
.../suites/inverted_index_p2/test_show_data.groovy | 26 ++++++++++++++++++++--
1 file changed, 24 insertions(+), 2 deletions(-)
diff --git a/regression-test/suites/inverted_index_p2/test_show_data.groovy
b/regression-test/suites/inverted_index_p2/test_show_data.groovy
index 5d86c1e7c05..6f798694550 100644
--- a/regression-test/suites/inverted_index_p2/test_show_data.groovy
+++ b/regression-test/suites/inverted_index_p2/test_show_data.groovy
@@ -774,6 +774,26 @@ suite("test_show_data_with_compaction", "p2") {
return data_size
}
+ def assert_show_data_size_close = { left_size, right_size, desc ->
+ double diff = Math.abs(left_size.toDouble() - right_size.toDouble())
+ double max_size = Math.max(left_size.toDouble(), right_size.toDouble())
+ double diff_ratio = diff / max_size
+ logger.info("{} show data size diff is {}, diff ratio is {}", desc,
diff, diff_ratio)
+ assertTrue(diff_ratio < 0.15d,
+ "${desc} show data size diff ratio ${diff_ratio} should be
less than 15%, " +
+ "diff: ${diff} KB, left size: ${left_size}, right size:
${right_size}")
+ }
+
+ def assert_documents_show_data_size_in_range = { data_size, desc ->
+ double size = data_size.toDouble()
+ double min_size = isCloudMode() ? 55.0d : 170.0d
+ double max_size = isCloudMode() ? 70.0d : 200.0d
+ logger.info("{} show data size is {}, expected range is [{}, {}],
isCloudMode: {}",
+ desc, size, min_size, max_size, isCloudMode())
+ assertTrue(size >= min_size && size <= max_size,
+ "${desc} show data size ${size} KB should be in [${min_size},
${max_size}] KB")
+ }
+
try {
set_be_config.call("inverted_index_compaction_enable", "true")
@@ -813,7 +833,9 @@ suite("test_show_data_with_compaction", "p2") {
assertTrue(another_with_index_size != "wait_timeout")
logger.info("with_index_size is {}, another_with_index_size is {}",
with_index_size, another_with_index_size)
- assertEquals(another_with_index_size, with_index_size)
+ assert_documents_show_data_size_in_range.call(with_index_size,
"documents table with inverted index compaction")
+ assert_documents_show_data_size_in_range.call(another_with_index_size,
"documents table without inverted index compaction")
+ assert_show_data_size_close.call(with_index_size,
another_with_index_size, "documents table")
set_be_config.call("inverted_index_compaction_enable", "true")
@@ -824,7 +846,7 @@ suite("test_show_data_with_compaction", "p2") {
def data_size_2 = create_table_run_compaction_and_wait(tableName)
logger.info("data_size_1 is {}, data_size_2 is {}", data_size_1,
data_size_2)
- assertEquals(data_size_1, data_size_2)
+ assert_show_data_size_close.call(data_size_1, data_size_2, "insert
table")
} finally {
// sql "DROP TABLE IF EXISTS ${tableWithIndexCompaction}"
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
