Probably it should be just "{}". If we're to use slf4j idioms, we should also remove some/most 'if (LOGGER.isDebugEnabled())' checks. Not yet, though.

On Mar 28, 2013, at 9:02 AM, magnayn <notifications@github.com> wrote:

> In src/main/mondrian/util/UtilCompatibleJdk14.java:
> 
> > @@ -144,7 +145,7 @@ public void cancelAndCloseStatement(Statement stmt) {
> >                  return;
> >              }
> >              if (LOGGER.isDebugEnabled()) {
> > -                LOGGER.debug(
> > +                LOGGER.debug("Error {}",
> Ah - I didn't realise the output was being read elsewhere. I assume it should be just "{}" ?
> 
> —
> Reply to this email directly or view it on GitHub.
>