[Fix][Common] Problems with comma-separated logic in variable values during variable substitution.(#9151)

Author: qifanlili

Diff for: seatunnel-common/src/main/java/org/apache/seatunnel/common/utils/PlaceholderUtils.java

@@ -17,6 +17,8 @@
 
 package org.apache.seatunnel.common.utils;
 
+import org.apache.commons.lang3.StringUtils;
+
 import java.util.regex.Matcher;
 import java.util.regex.Pattern;
 
@@ -28,6 +30,10 @@ public static String replacePlaceholders(String input, String placeholderName, S
 
     public static String replacePlaceholders(
             String input, String placeholderName, String value, String defaultValue) {
+        if (StringUtils.isNotBlank(value)) {
+            value = value.replaceAll("\"(.*?)\"", "$1");
+        }
+
         String placeholderRegex = "\\$\\{" + Pattern.quote(placeholderName) + "(:[^}]*)?\\}";
         Pattern pattern = Pattern.compile(placeholderRegex);
         Matcher matcher = pattern.matcher(input);

Diff for: seatunnel-common/src/test/java/org/apache/seatunnel/common/utils/PlaceholderUtilsTest.java

@@ -0,0 +1,38 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License.  You may obtain a copy of the License at
+ *
+ *    http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.seatunnel.common.utils;
+
+import org.junit.jupiter.api.Test;
+
+import lombok.extern.slf4j.Slf4j;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+@Slf4j
+public class PlaceholderUtilsTest {
+
+    @Test
+    public void testReplacePlaceholders() {
+        String input = "select ${columns} from table";
+        String placeholderName = "columns";
+        String value = "\"c_id,c_area_code,c_date\"";
+
+        String result = PlaceholderUtils.replacePlaceholders(input, placeholderName, value, null);
+        assertEquals("select c_id,c_area_code,c_date from table", result);
+    }
+}