The function issue is not the only problem (might not even be the most relevant one).

So, please follow Grants advice and post the actual execution plan.

And don't worry too much about Celko's post... If you'd take the time trying to read and understand what his real intention is, you'd probably have rewritten your code by then anyway... 😉

Beside he really knows SQL stuff (at least what rumour tells) he's got that "special" online personality. The bottom line: He doesn't like the way you name your columns, tables, and views. Doesn't change performance at all, but would help readability and maintainability, I admitt. But that's not the major concern here, I guess.... 😉

He's right about the DISTINCT issue though...