added logging to log command builder

This commit is contained in:
Sebastian Sdorra
2012-06-16 12:32:22 +02:00
parent f7cdf7aa07
commit f992b498a3

View File

@@ -38,6 +38,9 @@ package sonia.scm.repository.api;
import com.google.common.base.Objects; import com.google.common.base.Objects;
import com.google.common.collect.ImmutableList; import com.google.common.collect.ImmutableList;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import sonia.scm.cache.Cache; import sonia.scm.cache.Cache;
import sonia.scm.cache.CacheManager; import sonia.scm.cache.CacheManager;
import sonia.scm.repository.Changeset; import sonia.scm.repository.Changeset;
@@ -104,6 +107,12 @@ public final class LogCommandBuilder
/** name of the cache */ /** name of the cache */
private static final String CACHE_NAME = "sonia.scm.cache.log"; private static final String CACHE_NAME = "sonia.scm.cache.log";
/**
* the logger for LogCommandBuilder
*/
private static final Logger logger =
LoggerFactory.getLogger(LogCommandBuilder.class);
//~--- constructors --------------------------------------------------------- //~--- constructors ---------------------------------------------------------
/** /**
@@ -176,6 +185,11 @@ public final class LogCommandBuilder
if (disableCache) if (disableCache)
{ {
if (logger.isDebugEnabled())
{
logger.debug("get changeset for {} with disabled cache", id);
}
changeset = logCommand.getChangeset(id); changeset = logCommand.getChangeset(id);
} }
else else
@@ -185,6 +199,11 @@ public final class LogCommandBuilder
if (cpr == null) if (cpr == null)
{ {
if (logger.isDebugEnabled())
{
logger.debug("get changeset for {}", id);
}
changeset = logCommand.getChangeset(id); changeset = logCommand.getChangeset(id);
if (changeset != null) if (changeset != null)
@@ -195,11 +214,16 @@ public final class LogCommandBuilder
} }
else else
{ {
if (logger.isDebugEnabled())
{
logger.debug("get changeset {} from cache", id);
}
changeset = cpr.iterator().next(); changeset = cpr.iterator().next();
} }
} }
if (! disablePreProcessors && changeset != null) if (!disablePreProcessors && (changeset != null))
{ {
preProcessorUtil.prepareForReturn(repository, changeset); preProcessorUtil.prepareForReturn(repository, changeset);
} }
@@ -223,6 +247,11 @@ public final class LogCommandBuilder
if (disableCache) if (disableCache)
{ {
if (logger.isDebugEnabled())
{
logger.debug("get changesets for {} with disabled cache", request);
}
cpr = logCommand.getChangesets(request); cpr = logCommand.getChangesets(request);
} }
else else
@@ -233,6 +262,11 @@ public final class LogCommandBuilder
if (cpr == null) if (cpr == null)
{ {
if (logger.isDebugEnabled())
{
logger.debug("get changesets for {}", request);
}
cpr = logCommand.getChangesets(request); cpr = logCommand.getChangesets(request);
if (cpr != null) if (cpr != null)
@@ -240,9 +274,13 @@ public final class LogCommandBuilder
cache.put(key, cpr); cache.put(key, cpr);
} }
} }
else if (logger.isDebugEnabled())
{
logger.debug("get changesets from cache for {}", request);
}
} }
if (! disablePreProcessors && cpr != null) if (!disablePreProcessors && (cpr != null))
{ {
preProcessorUtil.prepareForReturn(repository, cpr); preProcessorUtil.prepareForReturn(repository, cpr);
} }
@@ -292,8 +330,7 @@ public final class LogCommandBuilder
* *
* @return {@code this} * @return {@code this}
*/ */
public LogCommandBuilder setDisablePreProcessors( public LogCommandBuilder setDisablePreProcessors(boolean disablePreProcessors)
boolean disablePreProcessors)
{ {
this.disablePreProcessors = disablePreProcessors; this.disablePreProcessors = disablePreProcessors;