[Optimization][client] explainSqlRecord 方法优化

[Optimization][client] explainSqlRecord 方法优化

dlink-client/dlink-client-1.11/src/main/java/com/dlink/executor/CustomTableEnvironmentImpl.java

@@ -250,9 +250,7 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             throw new TableException(
                 "Unsupported SQL query! explainSql() only accepts a single SQL query.");
         }
-        List<Operation> operationlist = new ArrayList<>(operations);
-        for (int i = 0; i < operationlist.size(); i++) {
-            Operation operation = operationlist.get(i);
+        Operation operation = operations.get(0);
         if (operation instanceof ModifyOperation) {
             record.setType("Modify DML");
         } else if (operation instanceof ExplainOperation) {
@@ -261,16 +259,13 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             record.setType("Query DML");
         } else {
             record.setExplain(operation.asSummaryString());
-                operationlist.remove(i);
             record.setType("DDL");
-                i = i - 1;
-            }
         }
         record.setExplainTrue(true);
-        if (operationlist.size() == 0) {
+        if ("DDL".equals(record.getType())) {
             return record;
         }
-        record.setExplain(planner.explain(operationlist, extraDetails));
+        record.setExplain(planner.explain(operations, extraDetails));
         return record;
     }
 

dlink-client/dlink-client-1.12/src/main/java/com/dlink/executor/CustomTableEnvironmentImpl.java

@@ -252,9 +252,7 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             throw new TableException(
                 "Unsupported SQL query! explainSql() only accepts a single SQL query.");
         }
-        List<Operation> operationlist = new ArrayList<>(operations);
-        for (int i = 0; i < operationlist.size(); i++) {
-            Operation operation = operationlist.get(i);
+        Operation operation = operations.get(0);
         if (operation instanceof ModifyOperation) {
             record.setType("Modify DML");
         } else if (operation instanceof ExplainOperation) {
@@ -263,16 +261,13 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             record.setType("Query DML");
         } else {
             record.setExplain(operation.asSummaryString());
-                operationlist.remove(i);
             record.setType("DDL");
-                i = i - 1;
-            }
         }
         record.setExplainTrue(true);
-        if (operationlist.size() == 0) {
+        if ("DDL".equals(record.getType())) {
             return record;
         }
-        record.setExplain(planner.explain(operationlist, extraDetails));
+        record.setExplain(planner.explain(operations, extraDetails));
         return record;
     }
 

dlink-client/dlink-client-1.13/src/main/java/com/dlink/executor/CustomTableEnvironmentImpl.java

@@ -253,9 +253,7 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             throw new TableException(
                 "Unsupported SQL query! explainSql() only accepts a single SQL query.");
         }
-        List<Operation> operationlist = new ArrayList<>(operations);
-        for (int i = 0; i < operationlist.size(); i++) {
-            Operation operation = operationlist.get(i);
+        Operation operation = operations.get(0);
         if (operation instanceof ModifyOperation) {
             record.setType("Modify DML");
         } else if (operation instanceof ExplainOperation) {
@@ -264,17 +262,14 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             record.setType("Query DML");
         } else {
             record.setExplain(operation.asSummaryString());
-                operationlist.remove(i);
             record.setType("DDL");
-                i = i - 1;
-            }
         }
         record.setExplainTrue(true);
-        if (operationlist.size() == 0) {
+        if ("DDL".equals(record.getType())) {
             //record.setExplain("DDL语句不进行解释。");
             return record;
         }
-        record.setExplain(planner.explain(operationlist, extraDetails));
+        record.setExplain(planner.explain(operations, extraDetails));
         return record;
     }
 

dlink-client/dlink-client-1.14/src/main/java/com/dlink/executor/CustomTableEnvironmentImpl.java

@@ -261,9 +261,8 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             throw new TableException(
                 "Unsupported SQL query! explainSql() only accepts a single SQL query.");
         }
-        List<Operation> operationlist = new ArrayList<>(operations);
-        for (int i = 0; i < operationlist.size(); i++) {
-            Operation operation = operationlist.get(i);
+
+        Operation operation = operations.get(0);
         if (operation instanceof ModifyOperation) {
             record.setType("Modify DML");
         } else if (operation instanceof ExplainOperation) {
@@ -272,17 +271,14 @@ public class CustomTableEnvironmentImpl extends TableEnvironmentImpl implements
             record.setType("Query DML");
         } else {
             record.setExplain(operation.asSummaryString());
-                operationlist.remove(i);
             record.setType("DDL");
-                i = i - 1;
-            }
         }
         record.setExplainTrue(true);
-        if (operationlist.size() == 0) {
+        if ("DDL".equals(record.getType())) {
             //record.setExplain("DDL语句不进行解释。");
             return record;
         }
-        record.setExplain(planner.explain(operationlist, extraDetails));
+        record.setExplain(planner.explain(operations, extraDetails));
         return record;
     }
 

dlink-client/dlink-client-1.15/src/main/java/com/dlink/executor/CustomTableEnvironmentImpl.java

@@ -214,9 +214,7 @@ public class CustomTableEnvironmentImpl extends AbstractStreamTableEnvironmentIm
             throw new TableException(
                 "Unsupported SQL query! explainSql() only accepts a single SQL query.");
         }
-        List<Operation> operationlist = new ArrayList<>(operations);
-        for (int i = 0; i < operationlist.size(); i++) {
-            Operation operation = operationlist.get(i);
+        Operation operation = operations.get(0);
         if (operation instanceof ModifyOperation) {
             record.setType("Modify DML");
         } else if (operation instanceof ExplainOperation) {
@@ -225,17 +223,14 @@ public class CustomTableEnvironmentImpl extends AbstractStreamTableEnvironmentIm
             record.setType("Query DML");
         } else {
             record.setExplain(operation.asSummaryString());
-                operationlist.remove(i);
             record.setType("DDL");
-                i = i - 1;
-            }
         }
         record.setExplainTrue(true);
-        if (operationlist.size() == 0) {
+        if ("DDL".equals(record.getType())) {
             //record.setExplain("DDL语句不进行解释。");
             return record;
         }
-        record.setExplain(planner.explain(operationlist, extraDetails));
+        record.setExplain(planner.explain(operations, extraDetails));
         return record;
     }