drew.allen (8/2/2012)

---

You're better off using a CTE than the CROSS APPLY.

WITH ActiveUpdates AS (

SELECT [ID], SortOrder, isActive, ROW_NUMBER() OVER( ORDER BY SortOrder DESC, [ID] ) - 1 AS rn

FROM @CustomerConditions

)

UPDATE...

This should work with your sample data.

select *

from CustomerConditions

where IsActive = 1

order by SortOrder

begin transaction

update CustomerConditions

set SortOrder = x.RowNum

from CustomerConditions c

cross apply

(

select ConditionID, ROW_NUMBER() over (order by isnull(SortOrder, 9999999), ConditionID)...

Lynn Pettis (8/1/2012)

---

Looking really good. You are on the right track for what you need to accomplish. I like the code you are writing, it is well structured,...

But the point was still that functionally "BETWEEN" in TSQL doesn't match what one would logically expect from the name. Which would be, "is value in the range defined by...

I think I understand what you are trying to do. Can you post some ddl (create table) and sample data (insert statements) in a consumable format?

I understand your point but the lowest value first makes sense. Would you tell somebody that you will come visit them sometime BETWEEN 7pm and 6pm? They would look at...

morepainot (8/1/2012)

---

Sean Lange (8/1/2012)

---

morepainot (8/1/2012)

---

Oh man looks like Im going to have trouble with this review tomorrow. I have a developer who is remotely looking at ny code but hes...

morepainot (8/1/2012)

---

Oh man looks like Im going to have trouble with this review tomorrow. I have a developer who is remotely looking at ny code but hes the type the...

•Do all staged tables exist in the source and in the stage database?

•Do all staged columns exist in the source and in the stage database?

•Is the data type of all...

Oh thats only there cause I have to create a table there. Im not sure what columns to add to it so I left it blank for the time being....

You need step away from...

You are quite welcome. Glad that worked!!! Thanks for letting us know.

morepainot (8/1/2012)

---

Sean Lange (8/1/2012)

---

morepainot (8/1/2012)

---

Should I do the same for this? I used your suggestion for the procedure and looks like the problem was solved. I have ONE more after...

morepainot (8/1/2012)

---

Should I do the same for this? I used your suggestion for the procedure and looks like the problem was solved. I have ONE more after this and I...

morepainot (8/1/2012)

---

[/color]

No, it means the procedure alread exists and you are trying to create it again. Two things you can do. One, change the CREATE PROCEDURE to ALTER...