Since there is no need for dynamic SQL in this case, just remove the quotes from the variable names... like this...

ALTER PROCEDURE spgetemp

@empname varchar(30),

@jdate datetime

AS

SET NOCOUNT ON

insert into #t exec...

Heh... Mom always said, "Never run with scissors"... being in a hurry on code has the same effect on careers.

Anyway, glad you figured it out... and thank for the feedback.

Awesome... thanks for the feedback, Derek!

Unless none of your dates have times on them, the original query and the replacement query are bad...

No...

You're missing the point... you will not get all of 01/04 using BETWEEN...

Not sure why you're whackin' at me on this one, Gail... real fact of the matter is that 4 million internal rows were generated and they don't show on the...

No... it's not perfect if you want to include the whole day of 01/04/2008...

No, they won't. At best an exec plan shows the number of rows that each operator accepted, processed and returned, but that's the number of rows in the result set,...

Heh... I use the Mil-spec Mark I Mod I Grey Matter Data Integrator... works everytime and any costs incurred can be simply resolved by a good night's sleep and maybe...

It's because you're converting the dates to text... try this, instead...

SELECT * FROM [MyDateTable]

WHERE StartDate >= '02/01/2007'

AND StartDate < '01/05/2008'

Notice that the "<" is in relation to the NEXT...

p.s. I just noticed that I left the code in that creates the "Raw2" table... that code is just not necessary and may be removed. Sorry for the...

Ok... sorry for the delay... couple of things came up and I couldn't get back to this... a shame, really, because this was kind'a fun.

In order to split the...

Did you use Clustered or Non-Clustered?

Check the spelling of @StartTime in the proc... notice... 2 "T's" in your exec... does it have 2 "T's" in the proc?