[fix](schema change) bugfix for light schema change while with rollup (#15681)

Describe your changes.
this problem come from pr: #11494

After add column to rollup index, it also change column UniqueId inside base index.

fe/fe-core/src/main/java/org/apache/doris/alter/SchemaChangeHandler.java

@@ -1106,18 +1106,20 @@ public class SchemaChangeHandler extends AlterHandler {
             hasPos = true;
         }
 
-        newColumn.setUniqueId(newColumnUniqueId);
+        // newColumn may add to baseIndex or rollups, so we need copy before change UniqueId
+        Column toAddColumn = new Column(newColumn);
+        toAddColumn.setUniqueId(newColumnUniqueId);
         if (hasPos) {
-            modIndexSchema.add(posIndex + 1, newColumn);
-        } else if (newColumn.isKey()) {
+            modIndexSchema.add(posIndex + 1, toAddColumn);
+        } else if (toAddColumn.isKey()) {
             // key
-            modIndexSchema.add(posIndex + 1, newColumn);
+            modIndexSchema.add(posIndex + 1, toAddColumn);
         } else if (lastVisibleIdx != -1 && lastVisibleIdx < modIndexSchema.size() - 1) {
             // has hidden columns
-            modIndexSchema.add(lastVisibleIdx + 1, newColumn);
+            modIndexSchema.add(lastVisibleIdx + 1, toAddColumn);
         } else {
             // value
-            modIndexSchema.add(newColumn);
+            modIndexSchema.add(toAddColumn);
         }
         LOG.debug("newColumn setUniqueId({}), modIndexSchema:{}", newColumnUniqueId, modIndexSchema);
     }

regression-test/data/rollup_p0/test_rollup_add_column.out

@@ -0,0 +1,7 @@
+-- This file is automatically generated. You should know what you did if you want to edit this
+-- !select_base --
+1	3	2	4	5
+
+-- !select_rollup --
+1	2	4
+

regression-test/suites/rollup_p0/test_rollup_add_column.groovy

@@ -0,0 +1,81 @@
+// 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.
+suite("test_rollup_add_column") {
+    def tbName = "test_rollup_add_column"
+    def rollupName = "test_rollup_add_column_index"
+
+    def getJobRollupState = { tableName ->
+        def jobStateResult = sql """  SHOW ALTER TABLE ROLLUP WHERE TableName='${tableName}' ORDER BY CreateTime DESC LIMIT 1; """
+        return jobStateResult[0][8]
+    }
+    def getJobColumnState = { tableName ->
+        def jobStateResult = sql """  SHOW ALTER TABLE COLUMN WHERE TableName='${tableName}' ORDER BY CreateTime DESC LIMIT 1; """
+        return jobStateResult[0][9]
+    }
+
+    sql "DROP TABLE IF EXISTS ${tbName}"
+    sql """
+            CREATE TABLE ${tbName} (
+                `k1` int(11) NOT NULL COMMENT "",
+                `k2` int(11) NOT NULL COMMENT "",
+                `v1` int(11) SUM NOT NULL COMMENT "",
+                `v2` int(11) SUM NOT NULL COMMENT ""
+            )
+            AGGREGATE KEY(`k1`, `k2`)
+            DISTRIBUTED BY HASH(`k1`) BUCKETS 1
+            PROPERTIES ("replication_num" = "1");
+        """
+
+    sql """ALTER TABLE ${tbName} ADD ROLLUP ${rollupName}(k1, v1);"""
+    int max_try_secs = 60
+    while (max_try_secs--) {
+        String res = getJobRollupState(tbName)
+        if (res == "FINISHED") {
+            break
+        } else {
+            Thread.sleep(2000)
+            if (max_try_secs < 1) {
+                println "test timeout," + "state:" + res
+                assertEquals("FINISHED",res)
+            }
+        }
+    }
+    Thread.sleep(2000)
+
+    sql "ALTER TABLE ${tbName} ADD COLUMN k3 INT KEY NOT NULL DEFAULT '3' AFTER k1 TO ${rollupName};"
+    max_try_secs = 60
+    while (max_try_secs--) {
+        String res = getJobColumnState(tbName)
+        if (res == "FINISHED") {
+            break
+        } else {
+            Thread.sleep(2000)
+            if (max_try_secs < 1) {
+                println "test timeout," + "state:" + res
+                assertEquals("FINISHED",res)
+            }
+        }
+    }
+
+    sql "insert into ${tbName} values(1, 2, 3, 4, 5);"
+
+    qt_select_base " select k1,k2,k3,v1,v2 from ${tbName} order by k1,k2,k3"
+    qt_select_rollup " select k1,k3,v1 from ${tbName} order by k1,k3"
+
+    sql "ALTER TABLE ${tbName} DROP ROLLUP ${rollupName};"
+    sql "DROP TABLE ${tbName} FORCE;"
+}