ddl: Reorganize Partition failure/rollback tests and fixes (pingcap#5…

…3544)

ref pingcap#45133

pkg/ddl/BUILD.bazel

@@ -256,7 +256,6 @@ go_test(
         "placement_policy_test.go",
         "placement_sql_test.go",
         "primary_key_handle_test.go",
-        "reorg_partition_test.go",
         "repair_table_test.go",
         "restart_test.go",
         "rollingback_test.go",
@@ -343,7 +342,6 @@ go_test(
         "//pkg/util/domainutil",
         "//pkg/util/gcutil",
         "//pkg/util/generic",
-        "//pkg/util/mathutil",
         "//pkg/util/mock",
         "//pkg/util/sem",
         "//pkg/util/sqlexec",

pkg/ddl/index.go

@@ -1560,6 +1560,14 @@ func (w *addIndexTxnWorker) checkHandleExists(idxInfo *model.IndexInfo, key kv.K
 	if hasBeenBackFilled {
 		return nil
 	}
+	if idxInfo.Global {
+		// 'handle' comes from reading directly from a partition, without partition id,
+		// so we can only compare the handle part of the key.
+		if ph, ok := h.(kv.PartitionHandle); ok && ph.Handle.Equal(handle) {
+			// table row has been back-filled already, OK to add the index entry
+			return nil
+		}
+	}
 	return ddlutil.GenKeyExistsErr(key, value, idxInfo, tblInfo)
 }