I know, it's horrible working with some of the data we get here! Been useful to get my head around the sp_executesql stuff tho. And yes, I got it working,...

The normalisation is the whole point of the exercise. I'm trying to import a flat file, with the same columns repeated 10 times, into a more logical structure - so...

Apologies for confusing things - the original post should have been the code below. I was going to run it in a loop and increment the counter field,

ALTER PROCEDURE [dbo].[procname]

AS

...

Thanks both of you. I replaced the line:

EXECUTE sp_Executesql @sqlquery, @ParamDefinition, @counter_txt

with:

EXECUTE(@SQLQuery)

and it ran. I obviously need to go and read up on this.

Thanks again