Ese tipo de consultas en inglés les llaman catch-all-queries o consultas para atrapar todo (no sé si me recuerda al Señor de los anillos o a Pokemon). El problema con las mismas es que no pueden generar un plan óptimo para la consulta, sino que tienen que utilizar un plan seguro que permita ser reutilizado en futuros usos de la misma consulta.
Tomemos un ejemplo usando la base de datos AdventureWorks (en mi caso probé con AdventureWorks2016). Es un caso muy similar al que tienes, pero con algo que la gente puede probar más fácilmente.
DECLARE @ProductId int = 855,
@TransactionType nchar(1) = NULL,
@Qty int = 2;
SELECT ProductID, TransactionID, Quantity
FROM Production.TransactionHistory
WHERE ( ProductID = @ProductID OR @ProductID IS NULL)
AND ( TransactionType = @TransactionType OR @TransactionType IS NULL)
AND ( Quantity = @Qty OR @Qty IS NULL);
Esta tabla tiene un índice en la columna ProductID, aunque no contiene las otras dos columnas. A pesar de que sería más eficiente hacer un index seek
sobre la columna ProductId y luego un Lookup
al índice clustered, no se puede utilizar esa opción por el riesgo de que en un futuro se pueda llegar a consultar con @ProductId como NULL y se tenga que realizar el lookup por todas las filas de la tabla. Por lo tanto, por seguridad siempre se hará un scan al índice clustered.
Para corregir eso, tenemos 2 opciones: usar RECOMPILE
o SQL dinámico.
Usar RECOMPILE, es algo muy sencillo de hacer, ya que sólo se agrega un par de cosas a la consulta.
DECLARE @ProductId int = 855,
@TransactionType nchar(1) = NULL,
@Qty int = 2;
SELECT ProductID, TransactionID, Quantity
FROM Production.TransactionHistory
WHERE ( ProductID = @ProductID OR @ProductID IS NULL)
AND ( TransactionType = @TransactionType OR @TransactionType IS NULL)
AND ( Quantity = @Qty OR @Qty IS NULL)
OPTION (RECOMPILE);
Así se puede generar un nuevo plan cada vez que se ejecute la consulta y puede ser óptimo puesto que no se reutilizará. La desventaja es que la instrucción se tiene que compilar cada vez que se ejecute y eso puede tomar tiempo.
Usar SQL Dinámico es un poco más complicado pero tiene la ventaja de poder reutilizar planes. Esta es una buena opción cuando las variaciones son pocas y no se tenderá a complicar demasiado la consulta.
DECLARE @ProductId int = NULL,
@TransactionType nchar(1) = NULL,
@Qty int = 2;
DECLARE @SQL nvarchar(MAX)
SET @SQL = N'
SELECT ProductID, TransactionType, Quantity
FROM Production.TransactionHistory ' + NCHAR(10)
+ STUFF( CASE WHEN @ProductId IS NOT NULL THEN N'AND ProductID = @ProductID ' + NCHAR(10) ELSE '' END
+ CASE WHEN @TransactionType IS NOT NULL THEN N'AND TransactionType = @TransactionType ' + NCHAR(10) ELSE '' END
+ CASE WHEN @Qty IS NOT NULL THEN N'AND Quantity = @Qty ' + NCHAR(10) ELSE '' END, 1, 3, 'WHERE') + ';'
--PRINT @SQL;
EXEC sp_executesql @SQL, N'@ProductId int, @TransactionType nchar(1), @Qty int', @ProductId, @TransactionType, @Qty;
Ambas opciones permitirán hacer el Index seek
y optimizarán la consulta, pero abusar de ellas puede ser un problema a largo plazo por el procesamiento adicional que implican. Por lo mismo, es recomendable evitar las consultas-atrapa-todo y dejarlas para casos muy específicos.
Respuesta basada en el artículo How to Confuse the SQL Server Query Optimizer
Parametro1 IS NULL
en realidad debiera ser@Parametro1 IS NULL
¿no?