Eliminate transaction control in the wrong places (these were added during debug of CopySpec)
Replace println warnings with logger.warn's
Remove extra commits (commit needs to happen centrally)
Wrap individual statements with commit_on_success

This test was failing because there were stealth transactions fouling
up the commit_on_success. It's not practical to wrap so much in a
commit_on_success, given the need to potentially close
statements/connections and do other work. It's likely transaction
management needs to be better specificed.