You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by yi...@apache.org on 2025/05/07 15:04:54 UTC

(doris) branch branch-2.1 updated: branch-2.1: [fix](systable) fix 'tables' systable's data_length filling error (#50595)

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

yiguolei pushed a commit to branch branch-2.1
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-2.1 by this push:
     new 3543124ed38 branch-2.1: [fix](systable) fix 'tables' systable's data_length filling error (#50595)
3543124ed38 is described below

commit 3543124ed38338759a45843e5aaf005a6b1d65ad
Author: Mingyu Chen (Rayner) <mo...@163.com>
AuthorDate: Wed May 7 23:04:46 2025 +0800

    branch-2.1: [fix](systable) fix 'tables' systable's data_length filling error (#50595)
    
    bp #50593
    Also fix a bug that the checking logic for 'TTableStatus' is not right.
    This bug only exists in branch-2.1
---
 be/src/exec/schema_scanner/schema_tables_scanner.cpp  |   2 +-
 .../data/query_p0/system/test_query_sys_tables.out    | Bin 6173 -> 6127 bytes
 .../query_p0/system/test_query_sys_tables.groovy      |  12 ++++++------
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/be/src/exec/schema_scanner/schema_tables_scanner.cpp b/be/src/exec/schema_scanner/schema_tables_scanner.cpp
index 8e711dd0132..ab513e8d951 100644
--- a/be/src/exec/schema_scanner/schema_tables_scanner.cpp
+++ b/be/src/exec/schema_scanner/schema_tables_scanner.cpp
@@ -239,7 +239,7 @@ Status SchemaTablesScanner::_fill_block_impl(vectorized::Block* block) {
         int64_t srcs[table_num];
         for (int i = 0; i < table_num; ++i) {
             const TTableStatus& tbl_status = _table_result.tables[i];
-            if (tbl_status.__isset.avg_row_length) {
+            if (tbl_status.__isset.data_length) {
                 srcs[i] = tbl_status.data_length;
                 datas[i] = srcs + i;
             } else {
diff --git a/regression-test/data/query_p0/system/test_query_sys_tables.out b/regression-test/data/query_p0/system/test_query_sys_tables.out
index 15e812c9eaf..8ff733239d8 100644
Binary files a/regression-test/data/query_p0/system/test_query_sys_tables.out and b/regression-test/data/query_p0/system/test_query_sys_tables.out differ
diff --git a/regression-test/suites/query_p0/system/test_query_sys_tables.groovy b/regression-test/suites/query_p0/system/test_query_sys_tables.groovy
index dd069816fd3..0eaae3a4fa3 100644
--- a/regression-test/suites/query_p0/system/test_query_sys_tables.groovy
+++ b/regression-test/suites/query_p0/system/test_query_sys_tables.groovy
@@ -181,13 +181,13 @@ suite("test_query_sys_tables", "query,p0") {
 
     // test tables
     // create test dbs
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName1}")
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName2}")
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName3}")
+    sql("DROP DATABASE IF EXISTS ${dbName1}")
+    sql("DROP DATABASE IF EXISTS ${dbName2}")
+    sql("DROP DATABASE IF EXISTS ${dbName3}")
     // create test tbs
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName1}")
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName2}")
-    sql("CREATE DATABASE IF NOT EXISTS ${dbName3}")
+    sql("CREATE DATABASE ${dbName1}")
+    sql("CREATE DATABASE ${dbName2}")
+    sql("CREATE DATABASE ${dbName3}")
     // create test tbs
     sql("use ${dbName1}")
     sql """


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