Cheers jmc - so in my case, as the code in post #5 is opening a connection which the Adapter WON'T close, I'd be better off getting rid of the 'cn_SQLConn.Open()' line, and letting the adapter take care of opening and closing the connection.

Have I got that right ?