I'm not a fan of splitting logging across separate logging calls like this.  I makes it harder to grep log files (which will become more of an issue if and when we start using logstash/graylog).

Btw, I had this idea that the `if (log.isDebugEnabled()) { }` pattern by overloading the logger class with calls that take a closure, which in turn returns a string, e.g.:
```
log.debug { some expensive to calculate log message }
```

One final thing - since this will only be called once per request, I wouldn't even bother putting the `isDebugEnabled()` check in there.  I would only use that in tightly nested bits of code that are called thousands of times.