Well, the naming is confusing. Commit and abort are, AFAIK, equivalent when dealing with read-only transactions. I thought abort sounded a little drastic, but I'm ok with changing to that if it makes the code easier to read.