should attempt rollback if commit fails?