Hjælp til lille stored procedure

Ikke testet ... problemet er dit dynamiske opbygning af dine query der ikke virker :-)

Sagt på en anden måde ... jeg antager hvad du har lavet der ikke virker :-)

Du burde kunne gøre sådan her:
SET ANSI_NULLS ON
GO
SET QUOTED_IDENTIFIER ON
GO
CREATE PROCEDURE Batch
    @TableName varchar(250),
    @LogTime datetime
AS
BEGIN
    SET NOCOUNT ON;

    DECLARE @BatchSize INT

    DECLARE @SQL varchar(500);
    SET @BatchSize = 10000


    WHILE @BatchSize <> 0
    BEGIN
        SET @SQL = 'DELETE TOP ' + @BatchSize + ' FROM ' + @TableName + ' WHERE LogTime < ' + @LogTime;
       
        EXEC @SQL;
       
        SET @BatchSize = @@rowcount
    END
END
GO

Tester

Jeg får følgende fejl:
Msg 245, Level 16, State 1, Procedure sp_batch_test, Line 16
Conversion failed when converting the varchar value 'DELETE TOP ' to data type int.

Følgende kald gav fejlen:
exec sp_batch_test 'tbl_data', '2012-12-11'

Kig på dette ... http://www.mssqltips.com/sqlservertip/1160/execute-dynamic-sql-commands-in-sql-server/

Så kan du angive parameters hvilket nok vil gøre det nemmere og mere sikkert mod sql injections.

Eventuelt kan du måske også ... TAGER INTET ANSVAR. Husk backup, test i dev og ikke i prod.

SET @SQL = 'DELETE TOP ' + CAST(@BatchSize AS VARCHAR) + ' FROM ' + @TableName + ' WHERE LogTime < ''' + CAST(@LogTime AS VARCHAR) + '''';

Sammen med overstående link, burde det kunne laves rimelig dejligt.

Hvis du ikke kan finde ud af det, så kan jeg måske kigge igen på det senere .... arbejdet kalder.

Takker - jeg leger videre :)

Det ser ud til at følgende virker rimeligt
SET @SQL = 'DELETE TOP (' + CAST(@BatchSize AS VARCHAR) + ') FROM ' + @TableName + ' WHERE LogTime < ''' + convert(nvarchar, @LogTime, 121) + '''';


Nu får jeg en anden fejl end konvtertering...
Msg 2812, Level 16, State 62, Procedure sp_batch, Line 20
Could not find stored procedure 'DELETE TOP (10000) FROM tbl_data WHERE LogTime < '2012-12-12 00:00:00.000''.

Sådan - ny lykkedes det - smid et svar

EXEC @SQL;
skulle ændres til
EXEC sp_executesql @SQL;

Ahh, det jeg havde glemt.

Svar.