/*
 * 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.shardingsphere.core.rewrite.placeholder;

import com.google.common.base.Joiner;
import lombok.Getter;
import lombok.RequiredArgsConstructor;

import java.util.LinkedList;
import java.util.List;

/**
 * Insert columns placeholder for rewrite.
 *
 * @author panjuan
 */
@RequiredArgsConstructor
@Getter
public final class InsertColumnsPlaceholder implements ShardingPlaceholder {
    
    private final boolean isPartColumns;
    
    private final List<String> columns = new LinkedList<>();
    
    @Override
    public String toString() {
        if (columns.isEmpty()) {
            return "";
        }
        if (isPartColumns) {
            return String.format(", %s", Joiner.on(", ").join(columns));
        }
        return String.format("(%s)", Joiner.on(", ").join(columns));
    }
}

@@ -978,41 +978,18 @@
 til.
-LinkedList;%0Aimport java.util.List
+Collection
 ;%0A%0A/
@@ -1197,23 +1197,28 @@
 nal 
-boolean isPartC
+Collection%3CString%3E c
 olum
@@ -1248,49 +1248,42 @@
 nal 
-List%3CString%3E columns = new LinkedList%3C%3E()
+boolean isToAppendCloseParenthesis
 ;%0A  
@@ -1414,18 +1414,31 @@
  (is
-PartColumn
+ToAppendCloseParenthesi
 s) %7B
@@ -1476,12 +1476,12 @@
 at(%22
-, 
+(
 %25s
+)
 %22, J
@@ -1546,33 +1546,32 @@
 ring.format(%22(%25s
-)
 %22, Joiner.on(%22, 

modify InsertColumnsPlaceholder