[fix][dingo-calcite] Add precision when the column type is char

Author: guojn1

dingo-calcite/src/main/codegen/templates/Parser.jj

@@ -6613,9 +6613,9 @@ SqlNode BuiltinFunctionCall() :
         <LPAREN> AddExpression(args, ExprContext.ACCEPT_SUB_QUERY) { aliasBuilder.append(this.token.image); }
         <AS> { aliasBuilder.append(" ").append(this.token.image); }
         (
-            dt = DataType() { args.add(dt); aliasBuilder.append(" ").append(this.token.image); }
+            dt = DataType() { args.add(dt); aliasBuilder.append(" ").append(dt.toString()); }
         |
-            <INTERVAL> { aliasBuilder.append(" ").append(this.token.image); } e = IntervalQualifier() { args.add(e); aliasBuilder.append(this.token.image); }
+            <INTERVAL> { aliasBuilder.append(" ").append(this.token.image); } e = IntervalQualifier() { args.add(e); aliasBuilder.append(e.toString()); }
         )
         <RPAREN> {
             SqlNode sqlNode = SqlStdOperatorTable.CAST.createCall(s.end(this), args);

dingo-calcite/src/main/java/io/dingodb/calcite/executor/ShowColumnsExecutor.java

@@ -102,7 +102,7 @@ private List<List<String>> getColumnFields() {
                 if (column.getPrecision() > 0) {
                     type = type + "(" + column.getPrecision() + ")";
                 }
-            } else if (type.equals("VARCHAR")) {
+            } else if (type.equals("VARCHAR") || type.equals("CHAR")) {
                 if (column.getPrecision() > 0) {
                     type = type + "(" + column.getPrecision() + ")";
                 }

dingo-calcite/src/main/java/io/dingodb/calcite/executor/ShowCreateTableExecutor.java

@@ -289,10 +289,10 @@ private String getCreateTable() {
                 if (!"TXN_LSM".equalsIgnoreCase(indexTable.getEngine())) {
                     createTableSqlStr.append(" engine=").append(indexTable.getEngine());
                 }
+                appendPart(indexTable, createTableSqlStr);
                 if (StringUtils.isNotBlank(indexTable.getComment())) {
-                    createTableSqlStr.append(" comment ").append(indexTable.getComment());
+                    createTableSqlStr.append(" comment ").append("'").append(indexTable.getComment()).append("'");
                 }
-                appendPart(indexTable, createTableSqlStr);
             }
         }
         createTableSqlStr.append("\r\n");

dingo-calcite/src/main/java/io/dingodb/calcite/executor/VariableValidator.java

@@ -52,7 +52,8 @@ public static String validator(String name, String value, String scope) {
         if (ScopeVariables.immutableVariables.contains(name)) {
             throw DingoErrUtil.newStdErr(ErrIncorrectGlobalLocalVar, name);
         }
-        if (("SESSION".equals(scope) || "USER".equals(scope)) && ScopeVariables.containsGlobalVariable(name.toLowerCase())) {
+        if (("SESSION".equals(scope) || "USER".equals(scope))
+            && ScopeVariables.containsGlobalVariable(name.toLowerCase())) {
             throw DINGO_RESOURCE.invalidSetGlobalVariable(name).ex();
         }
         if (name.equalsIgnoreCase("autocommit")

dingo-common/src/main/java/io/dingodb/common/mysql/scope/ScopeVariables.java

@@ -32,7 +32,7 @@ public final class ScopeVariables {
 
     public static final List<String> globalVariables = List.of("job_need_gc", "txn_history_duration",
         "safepoint_ts", "ssl_enable", "innodb_online_alter_log_max_size",
-        "max_allowed_packet", "table_definition_cache");
+        "max_allowed_packet", "table_definition_cache", "create_table_with_data");
 
     static {
         //immutableVariables.add("version_comment");