The CSS SQL Server Engineers have posted an interesting post, I myself don’t have any of the fake identity columns but I did many times suggest in newsgroups to use UPDLOCK and HOLDLOCK in a transaction to guarantee that 2 inserts would not generate the same key value. This stuff below is from the CSS SQL Server Engineers post here:

The following pattern typically stems from an old practice used in SQL 4.x and 6.x days, before IDENTITY was introduced.

begin tran 
declare @iVal int

select @iVal = iVal from CounterTable (HOLDLOCK) where CounterName = 'CaseNumber'

update CounterTable 
set iVal = @iVal + 1 
where CounterName = 'CaseNumber'

commit tran

return @iVal

This can be a dangerous construct. Assume that the query is cancelled (attention) right after the select. SQL Server treats this as a batch termination and does not execute other statements. The application now holds a lock under the open transaction and without proper handling it leads to blocking.

One Statement Fix

declare @iVal int

update CounterTable 
set @iVal = iVal = iVal + 1 
where CounterName = 'CaseNumber'

return @iVal

So what do you think? Will you use that in the future? Is this too dangerous/unfamiliar?

Take a look at this piece of code that was shown to me by Emtucifor in this forum post:

Important Note from Emtucifor:

The information in this section is outdated. It was true in a previous service pack of SQL 2000 but now performs as one would expect.

However, the point about this method being unreliable may still be true: please see the comments on this blog post for a new example of why the syntax:
@var = column = /expression/
does not always work as one might expect.

UPDATE #Temp SET  @T = Test1 = 99, @T = Test2 = 47

This proves that behind the scenes SQL Server expands the statement to the following:

UPDATE #Temp SET  @T = 99, @T = 47, Test1 = @T, Test2 = @T

Because of this proof that in fact the @Variable = Column = Expression syntax is not truly coupled in the way it appears to be (otherwise the first update would set Test1 to 99 and Test2 to 47 instead of both to 47), it is best to avoid that syntax entirely.

So what do you think? Good or might be prone to some kind of bug in the future and then what?

*** If you have a SQL related question try our Microsoft SQL Server Programming forum or our Microsoft SQL Server Admin forum