storagemgr: Use correct snapshot LSN in traces, and remove unused error return
This MR addresses some miscellaneous issues in the transaction manager:
- Removing an unused error return value from
updateCommittedEntry()
to simplify the callsite in the transaction manager'sBegin()
function. - Setting the snapshot LSN in the trace span after the critical section, to prevent a data race.