feat: moved to ON DELETE CASCADE for foreign keys

Author: fstagni

src/DIRAC/Core/Utilities/MySQL.py

@@ -1042,7 +1042,7 @@ def _createTables(self, tableDict, force=False):
                         # cmdList.append( '`%s` %s' % ( forTable, tableDict[forTable]['Fields'][forKey] )
                         cmdList.append(
                             "FOREIGN KEY ( `%s` ) REFERENCES `%s` ( `%s` )"
-                            " ON DELETE RESTRICT" % (key, forTable, forKey)
+                            " ON DELETE CASCADE" % (key, forTable, forKey)
                         )
 
                 engine = thisTable.get("Engine", "InnoDB")

src/DIRAC/WorkloadManagementSystem/DB/TaskQueueDB.py

@@ -319,18 +319,7 @@ def cleanOrphanedTaskQueues(self, connObj=False):
         if not orphanedTQs:
             return S_OK()
         orphanedTQs = [str(otq[0]) for otq in orphanedTQs]
-
-        for mvField in multiValueDefFields:
-            result = self._update(
-                f"DELETE FROM `tq_TQTo{mvField}` WHERE TQId in ( {','.join(orphanedTQs)} )", conn=connObj
-            )
-            if not result["OK"]:
-                return result
-
-        result = self._update(f"DELETE FROM `tq_TaskQueues` WHERE TQId in ( {','.join(orphanedTQs)} )", conn=connObj)
-        if not result["OK"]:
-            return result
-        return S_OK()
+        return self._update(f"DELETE FROM `tq_TaskQueues` WHERE TQId in ( {','.join(orphanedTQs)} )", conn=connObj)
 
     def __setTaskQueueEnabled(self, tqId, enabled=True, connObj=False):
         if enabled:
@@ -990,10 +979,6 @@ def deleteTaskQueueIfEmpty(self, tqId, tqOwner=False, tqOwnerGroup=False, connOb
         tqToDel = retVal["Value"]
 
         if tqToDel:
-            for mvField in multiValueDefFields:
-                retVal = self._update(f"DELETE FROM `tq_TQTo{mvField}` WHERE TQId = {tqId}", conn=connObj)
-                if not retVal["OK"]:
-                    return retVal
             retVal = self._update(f"DELETE FROM `tq_TaskQueues` WHERE TQId = {tqId}", conn=connObj)
             if not retVal["OK"]:
                 return retVal