drew.allen (6/5/2012)

---

That doesn't work Lynn, because the OrderDate is not in the GROUP BY, so it cannot be used in the ORDER BY clause unless it's in an aggregate function...

Sean Lange (6/5/2012)

---

WOW your datatypes are a disaster. I don't mean to sound insulting but these tables have so many problems it is hard to comment on all of them.

--------------------------

Table...

Other than changing this:

and DATEPART(YEAR, e.eecDateOfLastHire) = @yearpicked

to this:

and e.eecDateOfLastHire >= dateadd(yy, (@yearpicked - 1900), 0) and e.eecDateOfLastHire < dateadd(yy, (@yearpicked - 1900) + 1, 0)

Your code doesn't necessarily need...

David Moutray (6/5/2012)

---

Fair enough. You should do the work. I should do the work. However, we should not make others do the work. Define your acronyms...

David Moutray (6/5/2012)

---

An acronym is always unknown to someone. Think of the new guy who might never have heard or seen it before.

Also, Google is fallible. There are...

David Moutray (6/5/2012)

---

Oh, and you want to do that fifteen times per page while you're reading? What about corporate acronyms that don't exist on Google?

I know, I must have...

First, my favorite President is Theodore Roosevelt. After reading this little bit about President Lincoln though, I realize that if I were elected President (never happen) that I would...

Try this:

select

year(OrderDate) AS OrderYear,

MonthForTheYear = CASE WHEN MONTH(OrderDate)=1 THEN 'January'

...

David Moutray (6/5/2012)

---

Oh, and I absolutely despise abbreviations! What, are you too lazy to type it out?

What the *&(^% does YMMV stand for?

(See what...

When you run the stored procedure in SSMS do you run it as the same user that attempts to run it from the application?

LightVader (6/5/2012)

---

I didn't realize that you didn't need...

Just remember that this, and DATEPART(YEAR, e.eecDateOfLastHire) = @yearpicked, won't make use of an index on e.eecDateOfLastHire if it exists.

And the DDL for the tables and the execution plan.

Okay, missed this one: dbo.fn_FirstDayOfMonth.

Even that function has a function call, dbo.fn_NTDayOfWeek.