(shardingsphere) branch master updated: Refactor constructor of DorisShowQueryStatsStatement to empty buildAttributes (#38323)

[zhangliang]

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

zhangliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
     new 97b225e0158 Refactor constructor of DorisShowQueryStatsStatement to 
empty buildAttributes (#38323)
97b225e0158 is described below

commit 97b225e0158bb4a800e97a01f8a8cdafb5d5fee0
Author: Liang Zhang <zhangli...@apache.org>
AuthorDate: Tue Mar 3 23:53:59 2026 +0800

    Refactor constructor of DorisShowQueryStatsStatement to empty 
buildAttributes (#38323)
---
 .../statement/type/dml/DeleteStatementContextTest.java        |  2 +-
 .../doris/dal/show/DorisShowQueryStatsStatement.java          | 11 ++++++-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git 
a/infra/binder/core/src/test/java/org/apache/shardingsphere/infra/binder/context/statement/type/dml/DeleteStatementContextTest.java
 
b/infra/binder/core/src/test/java/org/apache/shardingsphere/infra/binder/context/statement/type/dml/DeleteStatementContextTest.java
index 088fcf02704..5f14708bec2 100644
--- 
a/infra/binder/core/src/test/java/org/apache/shardingsphere/infra/binder/context/statement/type/dml/DeleteStatementContextTest.java
+++ 
b/infra/binder/core/src/test/java/org/apache/shardingsphere/infra/binder/context/statement/type/dml/DeleteStatementContextTest.java
@@ -58,7 +58,7 @@ class DeleteStatementContextTest {
                 is(Arrays.asList("foo_tbl", "bar_tbl")));
     }
     
-    private static TableNameSegment createTableNameSegment(final String 
tableName) {
+    private TableNameSegment createTableNameSegment(final String tableName) {
         TableNameSegment result = new TableNameSegment(0, 0, new 
IdentifierValue(tableName));
         result.setTableBoundInfo(new TableSegmentBoundInfo(new 
IdentifierValue("foo_db"), new IdentifierValue("foo_schema")));
         return result;
diff --git 
a/parser/sql/statement/dialect/doris/src/main/java/org/apache/shardingsphere/sql/parser/statement/doris/dal/show/DorisShowQueryStatsStatement.java
 
b/parser/sql/statement/dialect/doris/src/main/java/org/apache/shardingsphere/sql/parser/statement/doris/dal/show/DorisShowQueryStatsStatement.java
index a3e3286251b..49e5f69b2b3 100644
--- 
a/parser/sql/statement/dialect/doris/src/main/java/org/apache/shardingsphere/sql/parser/statement/doris/dal/show/DorisShowQueryStatsStatement.java
+++ 
b/parser/sql/statement/dialect/doris/src/main/java/org/apache/shardingsphere/sql/parser/statement/doris/dal/show/DorisShowQueryStatsStatement.java
@@ -45,7 +45,7 @@ public final class DorisShowQueryStatsStatement extends 
DALStatement {
     
     private final boolean verbose;
     
-    private SQLStatementAttributes attributes;
+    private final SQLStatementAttributes attributes;
     
     public DorisShowQueryStatsStatement(final DatabaseType databaseType, final 
DatabaseSegment database, final FromTableSegment fromTable, final boolean all, 
final boolean verbose) {
         super(databaseType);
@@ -53,6 +53,11 @@ public final class DorisShowQueryStatsStatement extends 
DALStatement {
         this.fromTable = fromTable;
         this.all = all;
         this.verbose = verbose;
+        attributes = new SQLStatementAttributes(
+                new DatabaseSelectRequiredSQLStatementAttribute(),
+                new TableSQLStatementAttribute(null == fromTable ? null : 
fromTable.getTable()),
+                new TablelessDataSourceBroadcastRouteSQLStatementAttribute(),
+                new AllowNotUseDatabaseSQLStatementAttribute(true, null == 
database ? null : database.getIdentifier().getValue()));
     }
     
     /**
@@ -84,9 +89,5 @@ public final class DorisShowQueryStatsStatement extends 
DALStatement {
     
     @Override
     public void buildAttributes() {
-        String databaseName = null == database ? null : 
database.getIdentifier().getValue();
-        SimpleTableSegment table = null == fromTable ? null : 
fromTable.getTable();
-        attributes = new SQLStatementAttributes(new 
DatabaseSelectRequiredSQLStatementAttribute(), new 
TableSQLStatementAttribute(table),
-                new TablelessDataSourceBroadcastRouteSQLStatementAttribute(), 
new AllowNotUseDatabaseSQLStatementAttribute(true, databaseName));
     }
 }