gh-1646: review fixes

* simplify rollback loop
* add comments
* extract xlog_tx constructor/destructor logic into functions
* remoe xlog_tx_write() from xlog_close(): don't attempt to
write uncommitted transactions at shutdown