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 2013/05/06 14:25:58 UTC
svn commit: r1479544 - in
/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs:
JobManagerImpl.java TopologyCapabilities.java
Author: cziegeler
Date: Mon May 6 12:25:57 2013
New Revision: 1479544
URL: http://svn.apache.org/r1479544
Log:
Remove todo's for log statements
Modified:
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/TopologyCapabilities.java
Modified: sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java?rev=1479544&r1=1479543&r2=1479544&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java (original)
+++ sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/JobManagerImpl.java Mon May 6 12:25:57 2013
@@ -553,15 +553,15 @@ public class JobManagerImpl
*/
@Override
public void handleTopologyEvent(final TopologyEvent event) {
-// if ( this.logger.isDebugEnabled() ) { // TODO
- this.logger.info("Received topology event {}", event);
-// }
+ if ( this.logger.isDebugEnabled() ) {
+ this.logger.debug("Received topology event {}", event);
+ }
// check if there is a change of properties which doesn't affect us
if ( event.getType() == Type.PROPERTIES_CHANGED ) {
final Map<String, String> newAllInstances = TopologyCapabilities.getAllInstancesMap(event.getNewView());
if ( this.topologyCapabilities != null && this.topologyCapabilities.isSame(newAllInstances) ) {
- logger.info("No changes in capabilities."); // TODO - debug
+ logger.debug("No changes in capabilities.");
return;
}
}
Modified: sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/TopologyCapabilities.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/TopologyCapabilities.java?rev=1479544&r1=1479543&r2=1479544&view=diff
==============================================================================
--- sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/TopologyCapabilities.java (original)
+++ sling/trunk/bundles/extensions/event/src/main/java/org/apache/sling/event/impl/jobs/TopologyCapabilities.java Mon May 6 12:25:57 2013
@@ -127,7 +127,7 @@ public class TopologyCapabilities {
for(final InstanceDescription desc : view.getInstances() ) {
final String topics = desc.getProperty(JobConsumer.PROPERTY_TOPICS);
if ( topics != null && topics.length() > 0 ) {
- this.logger.info("Capabilities of {} : {}", desc.getSlingId(), topics); // TODO debug
+ this.logger.debug("Capabilities of {} : {}", desc.getSlingId(), topics);
for(final String topic : topics.split(",") ) {
List<InstanceDescription> list = newCaps.get(topic);
if ( list == null ) {