Originally posted by slimdave
This is a great way to get a snapshot too old error -- commiting within a cursor like this is very bad practice.
I would have thought it should be OK in this case since the table being updated is not in the cursor. Have I missed something?

Mind you, you will have a problem sorting out how much has been inserted and commited if it crashes in the middle . . .