> 1) You don't have to right-trim to get rid of trailing blanks on

varchars.

Initially we want to be as safe as possible. Until all is working.

> 2) You shouldn't need to cast the incoming CHARs as varchar. Try

it

> without and see if you can save some cpu time.

Same as 1.

> 2) You shouldn't be using field aliases here, as they are
> meaningless. All you want to do is pass values to variables.

That's

> extra cpu cycles you can avoid.

OK.

Anne Harrison seems to reckon that the Skip approach will not
improve anything plus. I can't get it to compile:

either
FOR SELECT SKIP :ROWS_PROCESSEDSOFAR FIRST :ROWS_PERCYCLE
or
FOR SELECT SKIP ROWS_PROCESSEDSOFAR FIRST ROWS_PERCYCLE

> Review the logic in the procedure body. I can see some problems in

it.


We were trying to find problems in this but could not. Maybe you
could enlighten us on the problems you see in the procedure body.


Regards,

Robert