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 e28029d2888 [branch-4.1][fix](test) relax show data size comparison
(#65927)
e28029d2888 is described below
commit e28029d2888cc14a6cff797c068fcfd2f9ace32e
Author: Dongyang Li <[email protected]>
AuthorDate: Fri Jul 24 11:13:11 2026 +0800
[branch-4.1][fix](test) relax show data size comparison (#65927)
### What problem does this PR solve?
Regression P2 build #203694 failed in both test_show_data and
test_show_data_multi_add. The logical data and row counts were correct,
but the suites required byte-for-byte equality between SHOW DATA sizes
produced through different table/index build paths. The observed
differences were about 13%, which matches the physical-size variance
already handled for test_show_data_with_compaction in #65387.
Jira: DORIS-27395
Build: http://172.20.48.17:8111/build/203694
Occurrences: 2000002151, 2000002153
### What is changed?
Replace the two cross-table exact size comparisons with the same 15%
ratio gate used by the recent compaction-suite fix. Row-count,
index-build, timeout, and non-cloud gates remain unchanged. Failure
messages now include both sizes, the absolute difference, and the ratio.
### Check List
- [x] git diff --check
- [ ] Rerun test_show_data and test_show_data_multi_add on branch-4.1
Regression P2 after merge.
---
.../suites/inverted_index_p2/test_show_data.groovy | 22 ++++++++++++++++++++--
1 file changed, 20 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 6f798694550..6121f288569 100644
--- a/regression-test/suites/inverted_index_p2/test_show_data.groovy
+++ b/regression-test/suites/inverted_index_p2/test_show_data.groovy
@@ -205,7 +205,16 @@ suite("test_show_data", "p2") {
load_httplogs_data.call(testTableWithIndex,
'test_httplogs_load_with_index', 'true', 'json', 'documents-1000.json')
def another_with_index_size =
wait_for_show_data_finish(testTableWithIndex, 60000, 0)
if (!isCloudMode()) {
- assertEquals(another_with_index_size, with_index_size)
+ double diff = Math.abs(another_with_index_size.toDouble() -
with_index_size.toDouble())
+ double max_size = Math.max(another_with_index_size.toDouble(),
with_index_size.toDouble())
+ double diff_ratio = diff / max_size
+ logger.info("show data size diff is {}, diff ratio is {}, with
index size is {}, " +
+ "another with index size is {}", diff, diff_ratio,
with_index_size,
+ another_with_index_size)
+ assertTrue(diff_ratio < 0.15d,
+ "show data size diff ratio ${diff_ratio} should be less
than 15%, " +
+ "diff: ${diff} KB, left size: ${with_index_size}, " +
+ "right size: ${another_with_index_size}")
}
} finally {
//try_sql("DROP TABLE IF EXISTS ${testTable}")
@@ -605,7 +614,16 @@ suite("test_show_data_multi_add", "p2") {
load_httplogs_data.call(testTableWithIndex,
'test_show_data_httplogs_multi_add_with_index', 'true', 'json',
'documents-1000.json')
def another_with_index_size =
wait_for_show_data_finish(testTableWithIndex, 60000, 0)
if (!isCloudMode()) {
- assertEquals(another_with_index_size, with_index_size2)
+ double diff = Math.abs(another_with_index_size.toDouble() -
with_index_size2.toDouble())
+ double max_size = Math.max(another_with_index_size.toDouble(),
with_index_size2.toDouble())
+ double diff_ratio = diff / max_size
+ logger.info("multi-add show data size diff is {}, diff ratio is
{}, with index size is {}, " +
+ "another with index size is {}", diff, diff_ratio,
with_index_size2,
+ another_with_index_size)
+ assertTrue(diff_ratio < 0.15d,
+ "multi-add show data size diff ratio ${diff_ratio} should
be less than 15%, " +
+ "diff: ${diff} KB, left size: ${with_index_size2}, " +
+ "right size: ${another_with_index_size}")
}
} finally {
//try_sql("DROP TABLE IF EXISTS ${testTable}")
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
