This is an automated email from the ASF dual-hosted git repository.

dataroaring 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 eda2818b95b [fix](group commit) Fix 
insert_group_commit_into_unique_sync_mode (#28283)
eda2818b95b is described below

commit eda2818b95bd4e4610ad7b213862909c55a040c5
Author: meiyi <myime...@gmail.com>
AuthorDate: Wed Dec 13 01:12:57 2023 +0800

    [fix](group commit) Fix insert_group_commit_into_unique_sync_mode (#28283)
---
 .../insert_group_commit_into_max_filter_ratio.groovy    |  6 +++++-
 .../insert_group_commit_into_unique_sync_mode.groovy    | 17 ++++++++++++++---
 2 files changed, 19 insertions(+), 4 deletions(-)

diff --git 
a/regression-test/suites/insert_p0/insert_group_commit_into_max_filter_ratio.groovy
 
b/regression-test/suites/insert_p0/insert_group_commit_into_max_filter_ratio.groovy
index 02f3ca1e7a9..51264d3d863 100644
--- 
a/regression-test/suites/insert_p0/insert_group_commit_into_max_filter_ratio.groovy
+++ 
b/regression-test/suites/insert_p0/insert_group_commit_into_max_filter_ratio.groovy
@@ -218,7 +218,11 @@ suite("insert_group_commit_into_max_filter_ratio") {
             sql """ set enable_insert_strict = false; """
             group_commit_insert """ insert into ${dbTableName} values (9, 'a', 
'a'); """, 0
         }
-        get_row_count_with_retry(4 + item == "nereids" ? 2 : 0)
+        if (item == "nereids") {
+            get_row_count_with_retry(6)
+        } else {
+            get_row_count_with_retry(4)
+        }
         order_qt_sql """ select * from ${dbTableName} """
     }
     sql """ truncate table ${tableName} """
diff --git 
a/regression-test/suites/insert_p0/insert_group_commit_into_unique_sync_mode.groovy
 
b/regression-test/suites/insert_p0/insert_group_commit_into_unique_sync_mode.groovy
index d278653ac1f..0d727b9ec44 100644
--- 
a/regression-test/suites/insert_p0/insert_group_commit_into_unique_sync_mode.groovy
+++ 
b/regression-test/suites/insert_p0/insert_group_commit_into_unique_sync_mode.groovy
@@ -22,10 +22,21 @@ suite("insert_group_commit_into_unique_sync_mode") {
     def tableName = "insert_group_commit_into_unique_sync"
     def dbTableName = dbName + "." + tableName
 
+    // For sync mode mode, the rows should be visible once the load is 
finished,
+    // but we meet publish_timeout sometimes
     def getRowCount = { expectedRowCount ->
-        def rowCount = sql "select count(*) from ${dbTableName}"
-        logger.info("rowCount: " + rowCount + ", expecedRowCount: " + 
expectedRowCount)
-        assertEquals(expectedRowCount, rowCount[0][0])
+        def retry = 0
+        while (retry < 30) {
+            if (retry > 0) {
+                sleep(1000)
+            }
+            def rowCount = sql "select count(*) from ${dbTableName}"
+            logger.info("rowCount: " + rowCount + ", retry: " + retry)
+            if (rowCount[0][0] >= expectedRowCount) {
+                break
+            }
+            retry++
+        }
     }
 
     def group_commit_insert = { sql, expected_row_count ->


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to