Pues no se que haras mal, pero igual te doy un par de sugerencias. Primero, es preferible utilizar un objeto de la clase StringBuilder de System.Text para hacer concatenaciones, con su metodo append, en vez de concatenar directamente a un string. Esto porque la operacion de concatenacion es una de las operaciones mas costosas en cuanto a consumo de recursos en el .NET Framework,
[
http://msdn.microsoft.com/library/de...et5_update.asp
http://support.microsoft.com/default...b;en-us;306822
http://support.microsoft.com/default...b;en-us;307340
]
Segundo, evita ante todo el uso de tipos de datos variant (tipo de dato que fue descontinuado en .NET, me extraña que todavia exista entre los tipos validos para los ADO.NET providers). Mira bien el tipo de datos del campo en la tabla y sustituyelo. Ahh, y si te es factible, utiliza Stored Procedures

(ya es mucho pedir, no??

)