You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by cz...@apache.org on 2015/08/10 15:07:44 UTC
svn commit: r1695057 - in
/sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal:
SlingServletResolver.java resource/ServletResourceProviderFactory.java
Author: cziegeler
Date: Mon Aug 10 13:07:44 2015
New Revision: 1695057
URL: http://svn.apache.org/r1695057
Log:
SLING-4939 : ServletResolver implementation is too verbose
Modified:
sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java
sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java
Modified: sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java?rev=1695057&r1=1695056&r2=1695057&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java (original)
+++ sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/SlingServletResolver.java Mon Aug 10 13:07:44 2015
@@ -1000,7 +1000,7 @@ public class SlingServletResolver
provider,
createServiceProperties(reference, provider));
- LOGGER.info("Registered {}", provider.toString());
+ LOGGER.debug("Registered {}", provider.toString());
synchronized (this.servletsByReference) {
servletsByReference.put(reference, new ServletReg(servlet, reg));
}
Modified: sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java?rev=1695057&r1=1695056&r2=1695057&view=diff
==============================================================================
--- sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java (original)
+++ sling/trunk/bundles/servlets/resolver/src/main/java/org/apache/sling/servlets/resolver/internal/resource/ServletResourceProviderFactory.java Mon Aug 10 13:07:44 2015
@@ -127,8 +127,8 @@ public class ServletResourceProviderFact
addByType(pathSet, ref);
if (pathSet.isEmpty()) {
- if (log.isInfoEnabled()) {
- log.info(
+ if (log.isDebugEnabled()) {
+ log.debug(
"create({}): ServiceReference has no registration settings, ignoring",
getServiceIdentifier(ref));
}
@@ -239,8 +239,8 @@ public class ServletResourceProviderFact
// SLING-512 only, set default methods if no extensions are declared
if (extensions == null || extensions.length == 0) {
- if (log.isInfoEnabled()) {
- log.info(
+ if (log.isDebugEnabled()) {
+ log.debug(
"addByType({}): No methods declared, assuming GET/HEAD",
getServiceIdentifier(ref));
}
@@ -248,8 +248,8 @@ public class ServletResourceProviderFact
}
} else if (methods.length == 1 && ALL_METHODS.equals(methods[0])) {
- if (log.isInfoEnabled()) {
- log.info("addByType({}): Assuming all methods for '*'",
+ if (log.isDebugEnabled()) {
+ log.debug("addByType({}): Assuming all methods for '*'",
getServiceIdentifier(ref));
}
methods = null;