fix(query): AsyncFunctionCall::display_name should respect quota (#18412)

During the semantic analysis phase of the `nextval` function, the logic for generating `display_name` might not have fully considered SQL identifier quoting rules.
If the sequence name was a reserved keyword or contained special characters, the generated `display_name` might not conform to SQL syntax, leading to inaccurate display or potential issues.

This change constructs the `display_name` by leveraging the `to_string()` method of the `ident` object returned by `normalize_identifier`.
The `ident.to_string()` method already incorporates the logic for necessary identifier quoting according to SQL rules.

In main:

```
root@localhost:8000/default/default> create or replace table mytest (seq int default nextval(`测试`));
error: APIError: QueryFailed: [1001]fail to parse default expr `CAST(nextval(测试) AS Int32 NULL)` (string length = 35) of field seq, SyntaxException. Code: 1005, Text = unable to recognize the rest tokens.
```

Author: TCeason

src/query/sql/src/planner/semantic/type_check.rs

@@ -5213,15 +5213,18 @@ impl<'a> TypeChecker<'a> {
             ))
             .set_span(span));
         }
-        let sequence_name = if let Expr::ColumnRef { column, .. } = arguments[0] {
+        let (sequence_name, display_name) = if let Expr::ColumnRef { column, .. } = arguments[0] {
             if column.database.is_some() || column.table.is_some() {
                 return Err(ErrorCode::SemanticError(
                     "nextval function argument identifier should only contain one part".to_string(),
                 )
                 .set_span(span));
             }
             match &column.column {
-                ColumnID::Name(ident) => normalize_identifier(ident, self.name_resolution_ctx).name,
+                ColumnID::Name(ident) => {
+                    let ident = normalize_identifier(ident, self.name_resolution_ctx);
+                    (ident.name.to_string(), format!("{}({})", func_name, ident))
+                }
                 ColumnID::Position(pos) => {
                     return Err(ErrorCode::SemanticError(format!(
                         "nextval function argument don't support identifier {}",
@@ -5245,7 +5248,6 @@ impl<'a> TypeChecker<'a> {
 
         databend_common_base::runtime::block_on(catalog.get_sequence(req))?;
 
-        let display_name = format!("{}({})", func_name, sequence_name);
         let return_type = DataType::Number(NumberDataType::UInt64);
         let func_arg = AsyncFunctionArgument::SequenceFunction(sequence_name);
 

tests/sqllogictests/suites/stage/sequence_as_default.test

@@ -2,10 +2,10 @@ statement ok
 create or replace sequence seq
 
 statement ok
-create or replace sequence seq1
+create or replace sequence `seq1_测试`
 
 statement ok
-create or replace table tmp (seq int default nextval(seq), a int, seq1 bigint not null default nextval(seq1));
+create or replace table tmp (seq int default nextval(seq), a int, seq1 bigint not null default nextval(`seq1_测试`));
 
 statement ok
 set sequence_step_size = 1;