Mejorar el SP, para actualizar si existe un registro, sino insertar

[karimbo]

Hola amigos, tengo este SP que me funciona bien, pero el tema es que me dijeron que no es performance y que debo mejorarlo.
Ya lo revisé y no le encuentro la vuelta, me pueden ayudar cómo debo mejorarlo.
La idea de este SP es si un Aula o Curso no tiene un horario definido entonces realizar un insert o de lo contrario update.

Código SQL:

CREATE PROCEDURE HORARIO_AULA_ACTUALIZAR_INSERTAR
@Aula_Id INT,
@Lunes BIT, 
@Martes BIT,
@Miercoles BIT,
@Jueves BIT,
@Viernes BIT,
@HoraDesde DATETIME,
@HoraHasta DATETIME
AS
BEGIN
    IF ((SELECT COUNT(hc.Id) FROM HORARIO_AULA ha WHERE ha.Aula_id =     @Aula_Id ANDha.Activo = 1) > 0)
    BEGIN
        UPDATE HORARIO_AULA SET 
               Lunes = @Lunes
              ,Martes = @Martes
              ,Miercoles = @Miercoles
              ,Jueves = @Jueves
              ,Viernes = @Viernes
              ,Sabado = @Sabado
              ,HoraDesde = @HoraDesde
              ,HoraHasta = @HoraHasta
         WHERE Aula_Id = @Aula_Id AND
               Activo = 1
    END
    ELSE
    BEGIN
        DECLARE @Maximo_Id BIGINT
        SELECT @Maximo_Id = ISNULL((COUNT(Id)+1), 1) FROM HORARIO_AULA
        
        INSERT INTO HORARIO_AULA (Id, Aula_Id, Lunes, Martes, Miercoles,   Jueves, Viernes, HoraDesde, HoraHasta, Activo)
                                
        VALUES (@Maximo_Id, @Aula_Id, @Domingo, @Lunes, @Martes, @Miercoles, @Jueves,  @Viernes, @HoraDesde, @HoraHasta, 1)
    END
END
 

Saludos,

[Libras]

Asi rapido lo unico que veo es que en la tabla de horario_aula tienes un proceso para asignar el maximo id, lo que te evitarias si usaras un campo identity en lugar de hacer eso, digo lo unico que haces es manejar un autonumerico.

Por otro lado cual es el plan de ejecucion de estos querys? quizas el performance vaya por el lado de indices, pero eso no se puede saber sin mostrar el plan, y otra cosa, este procedure como lo mandas llamar y cuantas veces se manda llamar???

[Malenko]

En SQL Server tienes la versión Merge que te hace eso mismo (insert si no existe el registro y sino un update).

[Libras]

usando merge o un update insert es lo mismo te maneja las mismas operaciones en ambos lados, solo que en un lado manejas menos codigo que en otro......

[uagrm]

Puede ocurrir un error de duplicate key al insertar en la tabla HORARIO_AULA si justo al momento de obtener el @Maximo_Id otro proceso obtiene el mismo Id.

[karimbo]

Gracias por sus respuestas, voy a hacer la modificación y utilizar un autoincremental para el campo Id, no tengo plan de ejecución, ahora estoy investigando Merge en sql.

[Libras]

Como que no tienes plan de ejecucion? el plan de ejecucion lo genera sql server, asi que una cosa es que no sepas que es eso y otra muy distinta que no tengas, para el merge que version de sql server tienes?

el plan de ejecucion lo puedes obtener ejecutando tu procedure o tu query con "ctrl+L"