Solved

Is Old SQL Server Table Trigger Performance Issue Workaround Still Valid?

Posted on 2013-12-09
3
262 Views
Last Modified: 2013-12-09
I am looking for on-line documentation or moral support!

Years ago, we had a developer that determined we had this problem with our SQL Server 2000 installation:   BUG: UPDATE Trigger May Have Long Parse and Compile Time

So he added the workaround code (line 4 and line 6):
CREATE TRIGGER [dbo].[ti_MtsEHistory] ON [dbo].[MtsEHistory] FOR INSERT AS
BEGIN

DECLARE @InsertedBugWorkAroundtable(EventID int)

	INSERT INTO @InsertedBugWorkAround SELECT EventID FROM inserted 

	-- Set the Created field for any rows being inserted.
	 UPDATE MtsEHistory
	    SET Created = GETDATE(),
		ModifyDateTime = GETDATE()
	   FROM MtsEHistory, inserted
	  WHERE MtsEHistory.EventID = inserted.EventID

END

Open in new window


When I joined the company we were still on SQL Server 2000.  I ran tests by removing the workaround code and could not recreate the performance problem.  

We are now on SQL Server 2008.  Regardless of whether the UPDATE statement in the code above needs refactoring, I need help proving to my supervisor that the workaround is no longer needed.  (He won't take my word for it.)  The MtsEHistory table has millions of row, lots of columns and is one of our most used tables.  It is common sense that any trigger code that doesn't do anything useful should be removed.
0
Comment
Question by:AEPRTD
3 Comments
 
LVL 9

Expert Comment

by:QuinnDex
ID: 39706904
first thing to do is find any other queries Sp's index's etc that reference the InsertedBugWorkAroundtable

removing the code or the whole trigger will only stop that table being updated any tests you did will still be using the table during execution and if its fundamental to the query will not be using new data
0
 
LVL 11

Accepted Solution

by:
John_Vidmar earned 500 total points
ID: 39706963
The link you provided describes the 4 conditions that must all exist for the problem to occur:
1. The trigger contains an UPDATE statement.
2. The UPDATE statement contains a JOIN clause.
3. Three (3) or more tables are listed in the JOIN clause.
4. Either the INSERTED or DELETED tables are listed in the JOIN clause.

Your trigger does not have 3 or more tables, therefore, no problem.  The workaround (if the problem did exist) was to replace the inserted table with a local table-variable, which was not done (i.e., line 12 should have the table-variable, not inserted).  I would eliminate the table-variable, if anything, its slowing down your trigger.
0
 

Author Comment

by:AEPRTD
ID: 39706973
I did not realize all four conditions had to exist for the bug to exist.  And as you stated, it wasn't just the act of declaring and populating a table variable that fixed the problem, it was the subsequent use of the table variable in joins that fixed the problem.
0

Featured Post

IT, Stop Being Called Into Every Meeting

Highfive is so simple that setting up every meeting room takes just minutes and every employee will be able to start or join a call from any room with ease. Never be called into a meeting just to get it started again. This is how video conferencing should work!

Join & Write a Comment

Everyone has problem when going to load data into Data warehouse (EDW). They all need to confirm that data quality is good but they don't no how to proceed. Microsoft has provided new task within SSIS 2008 called "Data Profiler Task". It solve th…
This article explains how to reset the password of the sa account on a Microsoft SQL Server.  The steps in this article work in SQL 2005, 2008, 2008 R2, 2012, 2014 and 2016.
Via a live example, show how to extract information from SQL Server on Database, Connection and Server properties
Via a live example, show how to setup several different housekeeping processes for a SQL Server.

760 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

19 Experts available now in Live!

Get 1:1 Help Now