It's useful in most cases to have the stack trace for debugging, but in some cases like this one it's not hard to see where the exception came from, and it's probably not useful to know who the caller is, so it just adds noise. I don't feel strongly either way. I can change it back if log.exception is preferred.