トリガーが挿入されていないすべての行を更新します

Question

私はここで何かが分かりません。このトリガーは、テーブル内のすべての行を更新しています（強制終了のパフォーマンス）。新しい挿入行で更新を実行するだけです。

CREATE TRIGGER [dbo].[update_location_topo_name] 
--fires at each row insert, queries topo map layer (must be present!) and inserts name of topo into new location record 
on [dbo].[TBL_LOCATIONS] 
after insert 
AS 
BEGIN 
update TBL_LOCATIONS 
set TOPO_NAME = dbo.QD24K_GRSM.NAME 
FROM dbo.tbl_locations 
inner join dbo.QD24K_GRSM 
on TBL_LOCATIONS.Location_ID = TBL_LOCATIONS.Location_ID 
WHERE (QD24K_GRSM.Shape.STContains(TBL_LOCATIONS.SHAPE) = 1) 
END 

Answer 1

挿入された行だけを取得するには、INSERTED疑似テーブルを参照する必要があります。

また、結合条件がTBL_LOCATIONS.Location_ID = TBL_LOCATIONS.Location_IDの場合は、まったく意味がありません。

おそらく、挿入の前に挿入する前に行を変更するトリガーとしてINSTEAD OFトリガーとしてこれを行う方がよいでしょう。

CREATE TRIGGER [dbo].[update_location_topo_name] 
ON [dbo].[TBL_LOCATIONS] 
INSTEAD OF INSERT 
AS 
    BEGIN 
     INSERT INTO TBL_LOCATIONS 
        (foo, 
        bar, 
        TOPO_NAME) 
     SELECT foo, 
      bar, 
      dbo.QD24K_GRSM.NAME 
     FROM INSERTED I 
      LEFT JOIN dbo.QD24K_GRSM 
       ON QD24K_GRSM.Shape.STContains(I.SHAPE) = 1/* Will insert additional 
                rows if more than one match*/ 
    END 

Answer 2

「挿入後」ではなく「挿入する」のトリガーを作成することをお勧めします。 これにより、実際にターゲット表に挿入する前に、必要に応じて新しく挿入された行を変更できます。 そして、 "挿入された"テーブルとターゲットテーブルを結合する必要は全くありません。このような

何か（実際にそのコードを実行しないような構文が間違っている可能性があります）：

CREATE TRIGGER [dbo].[update_location_topo_name] 
on [dbo].[TBL_LOCATIONS] 
Instead Of insert 
AS 
BEGIN 
    Insert Into TBL_LOCATIONS (...., TOPO_NAME) 
    Select ..., CrTable.NAME as TOPO_NAME 
     From Inserted Cross Apply 
     (
       Select top 1 QD24K_GRSM.NAME from QD24K_GRSM where 
       QD24K_GRSM.Shape.STContains(Inserted.SHAPE) = 1 
     ) as CrTable 
END