ISPN-4137 Transaction executed multiple times due to forwarded CommitCommand

https://issues.jboss.org/browse/ISPN-4137

This is only a partial fix. It's still possible for the commit to execute on a backup node before the rollback, but after the lock was released on the primary owner, see https://issues.jboss.org/browse/ISPN-2462.

* Remove the "skipTransactionCompletedCheck" transaction flag.
* Only mark the transaction as completed after running the prepare from
  state transfer (if needed).
* Do not try to roll back if commit fails and recovery is enabled.