Yeah moving this to an else has both semantic and readability benefits in my opinion. Semantically, if `self.get_success_response(doc)` ever raises a `SubmissionError` (which it shouldn't ever) that will be thrown; readability-wise, it communicates that this is not something we ever expect to fail and if we do expect it to fail at some point in the future it will require it's own error handling separate from `post_xform_to_couch`.

Between using an else and just putting it outside the try/except, I don't have a strong opinion; since all exceptions either return or raise, the two are equivalent.