MDEV-38716: Part 2

The last patch was incorrect - it did not consider columns with both

DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP

and these records would be re-written with new timestamps.

The fix is to add the ON UPDATE based default_fields back to the
destination table after the actual data copy.

Reviewed-by: TODO
Signed-off-by: Brandon Nesterenko <[email protected]>

Author: bnestere

sql/sql_table.cc

@@ -12720,16 +12720,6 @@ copy_data_between_tables(THD *thd, TABLE *from, TABLE *to,
           goto err;
         (copy_end++)->set(*ptr,def->field,0);
       }
-
-      /*
-        Ensure that TIMESTAMP or DATETIME fields with ON UPDATE clauses
-        have their default_fields copied, so they are executed after ALTER.
-      */
-      if (dfield_ptr && def->field->has_update_default_function())
-      {
-        *(dfield_ptr++)= *ptr;
-        ++to->s->default_fields;
-      }
     }
     else
     {
@@ -12746,13 +12736,16 @@ copy_data_between_tables(THD *thd, TABLE *from, TABLE *to,
       }
     }
   }
+
+  /*
+    Mark end of default field pointers. Note this is somewhat temporarily done,
+    as default_fields are re-considered later in the function for ON UPDATE
+    defaults (TIMESTAMP and DATETIME). That is, at the end of this function,
+    dfield_ptr may be overridden from this NULL reset to point to an ON UPDATE
+    field.
+  */
   if (dfield_ptr)
-  {
-    if (dfield_ptr == to->default_field)
-      to->default_field= NULL; // No default fields left
-    else
-      *dfield_ptr= NULL; // Mark end of default field pointers
-  }
+    *dfield_ptr= NULL;
 
   if (order)
   {
@@ -13114,6 +13107,34 @@ copy_data_between_tables(THD *thd, TABLE *from, TABLE *to,
     to->file->extra(HA_EXTRA_PREPARE_FOR_DROP);
   }
 
+  /*
+    To prevent this data copy from re-writing previously set default values,
+    this function previously has overridden to->default_fields to only account
+    for new fields added to the table (i.e those which should get new default
+    values). Now that the data copy is done, the to->default_field list is
+    missing fields that have ON UPDATE clauses (i.e. TIMESTAMP and DATETIME).
+    So restore these fields to the default_fields list.
+
+    Note the code is similar to the dfield_ptr modification earlier in the
+    funciton.
+  */
+  it.rewind();
+  for (Field **ptr=to->field ; *ptr ; ptr++)
+  {
+    def=it++;
+    if (def->field && dfield_ptr && def->field->has_update_default_function())
+    {
+      *(dfield_ptr++)= *ptr;
+    }
+  }
+  if (dfield_ptr)
+  {
+    if (dfield_ptr == to->default_field)
+      to->default_field= NULL; // No default fields left
+    else
+      *dfield_ptr= NULL; // Mark end of default field pointers
+  }
+
   DEBUG_SYNC(thd, "copy_data_between_tables_before_reset_backup_lock");
   if (backup_reset_alter_copy_lock(thd))
     error= 1;