This is an automated email from the ASF dual-hosted git repository.
hello-stephen pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 48d62f4161d [fix](regression) Wait row count before hot value analyze
(#63758)
48d62f4161d is described below
commit 48d62f4161d49a134757959e8aee7ab49ccd15d4
Author: yujun <[email protected]>
AuthorDate: Thu May 28 10:33:22 2026 +0800
[fix](regression) Wait row count before hot value analyze (#63758)
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary:
The hot value analyze regression test can run sample analyze before
Cloud table row count metadata is reported. In that state sample analyze
treats the table as empty and writes empty column statistics, making the
test flaky. This PR waits for SHOW DATA to report the inserted row count
before running analyze on non-empty test tables.
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [x] Regression test
- [ ] Unit Test
- [ ] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
- Behavior changed:
- [x] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [ ] Confirm the release note
- [ ] Confirm test cases
- [ ] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR
should merge into -->
Tests:
```
./build.sh --fe
./run-regression-test.sh --run -d statistics -s test_full_analyze_hot_value
git diff --check
```
---
.../statistics/test_full_analyze_hot_value.groovy | 32 ++++++++++++++++++++++
1 file changed, 32 insertions(+)
diff --git
a/regression-test/suites/statistics/test_full_analyze_hot_value.groovy
b/regression-test/suites/statistics/test_full_analyze_hot_value.groovy
index f2e5031eb19..ad7df099e8d 100644
--- a/regression-test/suites/statistics/test_full_analyze_hot_value.groovy
+++ b/regression-test/suites/statistics/test_full_analyze_hot_value.groovy
@@ -16,6 +16,35 @@
// under the License.
suite("test_full_analyze_hot_value") {
+ def wait_row_count_reported = { db, table, row, column, expected ->
+ def result = sql """show frontends;"""
+ logger.info("show frontends result origin: " + result)
+ def host
+ def port
+ for (int i = 0; i < result.size(); i++) {
+ if (result[i][8] == "true") {
+ host = result[i][1]
+ port = result[i][4]
+ }
+ }
+ def tokens = context.config.jdbcUrl.split('/')
+ def url = tokens[0] + "//" + host + ":" + port
+ logger.info("Master url is " + url)
+ connect(context.config.jdbcUser, context.config.jdbcPassword, url) {
+ sql """use ${db}"""
+ result = sql """show frontends;"""
+ logger.info("show frontends result master: " + result)
+ for (int i = 0; i < 120; i++) {
+ Thread.sleep(5000)
+ result = sql """SHOW DATA FROM ${table};"""
+ logger.info("result " + result)
+ if (result[row][column] == expected) {
+ return;
+ }
+ }
+ throw new Exception("Row count report timeout.")
+ }
+ }
sql """drop database if exists test_full_analyze_hot_value"""
sql """create database test_full_analyze_hot_value"""
@@ -37,6 +66,7 @@ suite("test_full_analyze_hot_value") {
"""
// Insert 100 rows: value1 has 2 values, "0" and "1", each appearing 50
times
sql """insert into full_hot_skew select number, number % 2 from
numbers("number"="100")"""
+ wait_row_count_reported("test_full_analyze_hot_value", "full_hot_skew", 0,
4, "100")
sql """analyze table full_hot_skew with sync"""
def result = sql """show column stats full_hot_skew(value1)"""
@@ -94,6 +124,7 @@ suite("test_full_analyze_hot_value") {
)
"""
sql """insert into full_hot_special select number, " : ;a" from
numbers("number"="100")"""
+ wait_row_count_reported("test_full_analyze_hot_value", "full_hot_special",
0, 4, "100")
sql """analyze table full_hot_special with sync with hot value"""
result = sql """show column stats full_hot_special(value1)"""
@@ -173,6 +204,7 @@ suite("test_full_analyze_hot_value") {
)
"""
sql """insert into full_hot_all_null select number, null from
numbers("number"="100")"""
+ wait_row_count_reported("test_full_analyze_hot_value",
"full_hot_all_null", 0, 4, "100")
sql """analyze table full_hot_all_null with sync with hot value"""
result = sql """show column stats full_hot_all_null(value1)"""
logger.info("Test9 all-null result: " + result)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
