1) What is the point of this (isn't PRIMARY KEY sufficient by itself and the UNIQUE redundant?):

PRIMARY KEY (ID),

UNIQUE (ID))

2) I can probably count on 2 hands the number of times in nearly 20 years of SQL server work where one or more indexes on a temp table have IMPROVED performance of a sproc overall. I cannot count the number of times I have REMOVED them in client's code to make things FASTER.

3) "The primary reason is that using a #temp table allows normal procedure compilation and caching, whereas table variables (@temp) may force a recompile every time a procedure is run." - incorrect on several points

4) " table variables do not update statistics" - nothing to do with updating stats, the limitation concerns whether or not they are created

5) "or the transaction log so they will fail if used in a TRANS/ROLLBACK TRANS scenario" - incorrect. Actually this leads to one of the few scenarios where they are beneficial, namely keeping data in place on rollback to subsequently use for another purpose such as auditing or debugging.