>Hi Naomi.
>This system is inherited from an other programmer and the least amont of change the better .......They are using OLEDB and the least amount of chnges the better......
>I'll try out your changes but is there any reason why using the Temporary table would not work.
>I think its easier to follow putting code into small chunks rather than 1 large piece of code
>
>regards,
>Gerard
>
>
>
This particular statement doesn't strike me as complex in the way I re-wrote it. If you're using SQL Server 2005 and up, then you can use CTE instead of derived table.

In any case, the first portion of the code (that invokes first ExecuteReader) should be commented out.

Try my changes (and use parameters) and let me know how does it work.