Timeout al crear Procedimiento Almacenado en local

[gnzsoloyo]

En mi opinión el SP está haciendo demasiadas tareas inutilmente.
Básicamente (y esta es la forma que manejo algunas con el mismo objetivo), esto se puede resolver con un SP que ejecuta tres sentencias y un trigger para poner los valores de subítem (eso si el valor no viene en la tabla temporal misma).
La sintaxis de este ejemplo recibe tres parámetros, devolviendo en el tercero un valor mayor a cero si se pudo realizar, o bien cero sino se pudo, ya que devuelve el LAST_INSERT_ID().
Nota: Estoy suponiendo, para este caso, que los campos de la tabla destino se llaman igual que en la tabla origen.

Código MySQL:

DELIMITER $$
 
CREATE PROCEDURE sp_insertarPresupuesto (IN idCliente INT, IN fechaAlta DATETIME, INOUT idPresupuesto INT)
 
-- iniciamos el Procedimiento
 
BEGIN
 
    -- insertamos la cabecera del presupuesto
    INSERT INTO LM_Presupuestos values ('',idCliente, FechaAlta);
 
    -- recuperamos el idPresupuesto generado
    SET idPresupuesto = LAST_INSERT_ID();
  IF idPresupuesto > 0 THEN
    -- capturamos los datos de la tabla temporal
    INSERT INTO LM_detallePresupuesto(
      idPresupuesto,
      idCliente,
      Referencia,
      Detalle,
      IVA,
      PrecioUnitario,
      Cantidad)
    SELECT
      idPresupuesto,
      idCliente,
      Referencia,
      Detalle,
      IVA,
      PrecioUnitario,
      Cantidad
    FROM LM_tmpPresupuestos;
    -- se vacía la tabla temporal
      TRUNCATE TABLE LM_tmpPresupuestos;
    END IF;
END
 
$$
 
DELIMITER;
 

Si los valores de subitem recibidos no vienen de la tabla origen, entonces hay que poner un TRIGGER similar a este:

Código MySQL:

CREATE TRIGGER SET_SUBITEM_PRESUPUESTOS BEFORE INSERT
ON  LM_detallePresupuesto
FOR EACH ROW
BEGIN
        IF(SELECT MAX(SUBITEM) FROM  LM_detallePresupuesto WHERE idPresupuesto=NEW.idPresupuesto) IS NULL THEN
      SET NEW.SUBITEM = 1;
    ELSE
      SET NEW.SUBITEM = (SELECT MAX(SUBITEM) FROM  LM_detallePresupuesto WHERE idPresupuesto=NEW.idPresupuesto) + 1;
    END IF;
END$$
 
DELIMITER ;
 

Obviamente, este script hace referencia a un campo "SUBITEM" que debería existir, o llamarse de otra forma.