To Max,

Can you post the complete stored procedure.

Thanks

SET @running=1

1. Your WHILE loop already contains:

insert into @xp_results
execute master.dbo.xp_sqlagent_enum_jobs 1, @job_owner, @job_id

set @running= (SELECT top 1 running from @xp_results)

So, you can simply add ‘SET @running=0′ immediately before entering the WHILE loop and let the loop take care of this ‘insert exec’ and ’set’ portion.

Otherwise, you are essentially simulating the first “loop” of the WHILE loop outside the loop, and that’s just plain loopy.

2. Because you’re now relying on your loop to do what it’s supposed to do, you can remove the first ‘WAITFOR DELAY’ statement and @seccount increment (the one outside the loop), as we’ve got one baked into our loop already.

3. I had to use the following to identify my job in the results of xp_sqlagent_enum_jobs:

INSERT INTO @Results
EXECUTE master.dbo.xp_sqlagent_enum_jobs 1, @JobOwnerName
SET @IsRunning = (SELECT TOP 1 running FROM @Results WHERE job_id=@JobId)