Massive.cs should not be using @@IDENTITY and should use SCOPE_IDENTITY() instead #78

Closed
mattmc3 opened this Issue Aug 16, 2011 · 3 comments

Projects

None yet

3 participants

@mattmc3

@@IDENTITY in line 382 should be replaced with SCOPE_IDENTITY(), as @@IDENTITY is an unreliable way of returning an IDENTITY key from newly inserted row. @@IDENTITY can cause subtle bugs, especially in a case where triggers are used. http://blogs.msdn.com/b/spike/archive/2009/10/20/my-take-on-identity-vs-scope-identity.aspx

@KevinHoward

It appears that this issue has been submitted before.

#18

@mattmc3
@robconery
Collaborator

You can change as needed. Going to keep in place for SQL CE support...

@robconery robconery closed this Sep 19, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment