This isn't a change you made, but I was looking at this, where does this bubble up? We're logging a message, not a stack trace, then re-throwing the exception. Does the stacktrace ever get logged? If so we now have a double log for the same error, if not we have an issue because we won't know what went wrong. Something we can fix separately, but it seems bad.