日期时间比较更新两列

Question

当新作业添加到数据库时，我有一个由 SQL Server 创建的日期时间。将作业添加到数据库后 3 小时且尚未完成，应更改具有唯一作业 ID 的作业的日志级别和状态。

这是我到目前为止所拥有的。 （不检查状态）

use DistributedComputing
if exists (select * from sys.objects where type = 'P' and name = 'proc_dc_timeout')
drop procedure proc_dc_timeout
go
create procedure proc_dc_timeout
AS
declare @JobID int
begin
if exists (select StartDate from Job where JobID = @JobID and DATEDIFF(hh,StartDate,GETDATE()) > 3 )
update Job
set LogLevel = 1, Status = 'error'
end

go
execute proc_dc_timeout

但是如果我运行查询，则不会影响任何行。请帮忙

Answer 1

您的变量 @JobID 已声明，但未设置 - 因此 - 您的 IF 语句永远不会成功，因为与 NULL 值的任何比较都将返回 false。您需要将值分配给变量，或者使用 IS NULL 比较

Answer 2

正如上面的答案所指出的，JobID 为空。假设您想要检查和更新所有作业，这可能就是您想要的：

create procedure proc_dc_timeout
AS
update Job
set LogLevel = 1, Status = 'error'
where DATEDIFF( minute, StartDate, GETDATE() ) > 180
    and Status not in ( 'error' )

我使用 DATEDIFF(分钟...) 而不是小时，但这只是个人偏好，以确保关闭-一个错误不会造成如此大的差异。

Answer 3

有几件事......

您的 @JobID 变量似乎应该是过程的参数。你永远不会给这个变量赋值。

其次，您的 DATEDIFF(hh, StartDate, GETDATE()) > 3 最有可能更改为 DATEDIFF(hh, StartDate, GETDATE()) >= 3;正如目前所写，在达到 4 小时标记之前，您不会进行任何更新。

此外，您是否考虑过使用 ALTER PROCEDURE 语法，而不是在每次运行时检查过程是否存在并删除/重新创建？

ALTER PROCEDURE proc_dc_timeout ( @JobID INT )
AS
    BEGIN
        UPDATE j
        SET j.[LogLevel = 1,
            j.[Status] = 'error'
        FROM [Job] j
        WHERE j.[JobID] = @JobID
            AND DATEDIFF(hh, j.[StartDate], GETDATE()) >= 3
    END
GO

Answer 4

这将不起作用，因为您没有为 @JobID 设置任何值，它将为 null 并且您存在的查询将始终失败

Answer 5

执行过程时您没有给 @JobID 赋值，并且参数声明位于错误的位置。

CREATE PROCEDURE proc_dc_timeout
   @JobID int // <== declare parameter here
AS
BEGIN
    //...
END
go

EXECUTE proc_dc_timeout @JobID // <== put value here instead @JobID