See #24194 (comment) via @bdarnell:

EndTransaction currently declares the abort span for read/write whether it's a commit or abort:

Although I think you're right that it doesn't actually need to unless it's an abort and we're resolving local intents.