svn commit: r1134317 - in /juddi/trunk/juddi-core/src/main/java/org/apache/juddi: subscription/SubscriptionNotifier.java subscription/notify/NotifierFactory.java validation/ValidatePublish.java

[ks...@apache.org]

Author: kstam
Date: Fri Jun 10 13:25:17 2011
New Revision: 1134317

URL: http://svn.apache.org/viewvc?rev=1134317&view=rev
Log:
JUDDI-495, toning down logging

Modified:
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/notify/NotifierFactory.java
    juddi/trunk/juddi-core/src/main/java/org/apache/juddi/validation/ValidatePublish.java

Modified: juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java
URL: http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java?rev=1134317&r1=1134316&r2=1134317&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java (original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/SubscriptionNotifier.java Fri Jun 10 13:25:17 2011
@@ -125,7 +125,7 @@ public class SubscriptionNotifier extend
 		if ((firedOnTime(scheduledExecutionTime()) || alwaysNotify) && registryMayContainUpdates()) {
 			long startTime = System.currentTimeMillis();
 			desiredDate = null;
-			log.info("Start Notification background task; checking if subscription notifications need to be send out..");
+			log.debug("Start Notification background task; checking if subscription notifications need to be send out..");
 			
 			Collection<Subscription> subscriptions = getAllAsyncSubscriptions();
 			for (Subscription subscription : subscriptions) {
@@ -143,10 +143,10 @@ public class SubscriptionNotifier extend
 							publisher.setAuthorizedName(subscription.getAuthorizedName());
 							SubscriptionResultsList resultList = subscriptionImpl.getSubscriptionResults(getSubscriptionResults, publisher);
 							if (resultListContainsChanges(resultList)) {
-								log.info("We have a change and need to notify..");
+								log.debug("We have a change and need to notify " + subscription.getSubscriptionKey());
 								notify(getSubscriptionResults,resultList, notificationDate);
 							} else {
-								log.info("No changes where recorded, no need to notify.");
+								log.debug("No changes where recorded, no need to notify.");
 							}
 						}
 					} catch (Exception e) {
@@ -218,7 +218,7 @@ public class SubscriptionNotifier extend
 			if (log.isDebugEnabled()) log.debug("Period " + period.getStartPoint() + " " + period.getEndPoint());
 			getSubscriptionResults.setCoveragePeriod(period);
 		} else {
-			log.info("Client does not yet want a notification. The next desidered notification Date " + nextDesiredNotificationDate + ". The current interval [" 
+			log.debug("Client does not yet want a notification. The next desidered notification Date " + nextDesiredNotificationDate + ". The current interval [" 
 				+ startPoint + " , " + endPoint + "] therefore skipping this notification cycle.");
 			if (desiredDate==null || nextDesiredNotificationDate.getTime() < desiredDate.getTime()) {
 				desiredDate = nextDesiredNotificationDate;

Modified: juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/notify/NotifierFactory.java
URL: http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/notify/NotifierFactory.java?rev=1134317&r1=1134316&r2=1134317&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/notify/NotifierFactory.java (original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/subscription/notify/NotifierFactory.java Fri Jun 10 13:25:17 2011
@@ -35,7 +35,8 @@ public class NotifierFactory {
 			}
 		}
 		if (notifierClassName == null) {
-			log.info("The bindingTemplate " + bindingTemplate.getEntityKey() + " does not contain a tModel to define its type of transport. Defaulting " 
+			//JUDDI-496 TODO make sure the tModel is loaded
+			log.debug("The bindingTemplate " + bindingTemplate.getEntityKey() + " does not contain a tModel to define its type of transport. Defaulting " 
 				  +	"to http.");
 			notifierClassName = "org.apache.juddi.subscription.notify.HTTPNotifier";
 		}

Modified: juddi/trunk/juddi-core/src/main/java/org/apache/juddi/validation/ValidatePublish.java
URL: http://svn.apache.org/viewvc/juddi/trunk/juddi-core/src/main/java/org/apache/juddi/validation/ValidatePublish.java?rev=1134317&r1=1134316&r2=1134317&view=diff
==============================================================================
--- juddi/trunk/juddi-core/src/main/java/org/apache/juddi/validation/ValidatePublish.java (original)
+++ juddi/trunk/juddi-core/src/main/java/org/apache/juddi/validation/ValidatePublish.java Fri Jun 10 13:25:17 2011
@@ -322,7 +322,6 @@ public class ValidatePublish extends Val
 		if (config==null) {
 			try {
 				config = AppConfig.getConfiguration();
-				log.info("Configuration:" + config);
 			} catch (ConfigurationException ce){
 				log.error("Could not optain config. " + ce.getMessage(), ce);
 			}



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@juddi.apache.org
For additional commands, e-mail: commits-help@juddi.apache.org