wink-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jra...@apache.org
Subject svn commit: r1027164 - in /incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client: ClientConfig.java RestClient.java
Date Mon, 25 Oct 2010 15:17:54 GMT
Author: jramos
Date: Mon Oct 25 15:17:53 2010
New Revision: 1027164

URL: http://svn.apache.org/viewvc?rev=1027164&view=rev
Log:
Minor logging updates for wink-client

Modified:
    incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/ClientConfig.java
    incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/RestClient.java

Modified: incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/ClientConfig.java
URL: http://svn.apache.org/viewvc/incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/ClientConfig.java?rev=1027164&r1=1027163&r2=1027164&view=diff
==============================================================================
--- incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/ClientConfig.java
(original)
+++ incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/ClientConfig.java
Mon Oct 25 15:17:53 2010
@@ -465,7 +465,9 @@ public class ClientConfig implements Clo
                 String supportDTD = System.getProperty(WINK_SUPPORT_DTD_EXPANSION, String.valueOf(WINK_CLIENT_SUPPORT_DTD_EXPANSION_DEFAULT));
                 boolean toSet = Boolean.valueOf(supportDTD);  // require "true" or "false",
not "yes" or "no" or other variants (see parseBoolean vs. valueOf javadoc)
                 properties.put(WINK_SUPPORT_DTD_EXPANSION, String.valueOf(toSet));
-                logger.trace("Wink client readTimeout default value is {}.", String.valueOf(toSet));
//$NON-NLS-1$
+                if(logger.isTraceEnabled()) {
+                    logger.trace("Wink client readTimeout default value is {}.", String.valueOf(toSet));
//$NON-NLS-1$
+                }
             } catch (Exception e) {
                 logger.trace("Error processing {} system property: {}", WINK_SUPPORT_DTD_EXPANSION,
e); //$NON-NLS-1$
             }

Modified: incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/RestClient.java
URL: http://svn.apache.org/viewvc/incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/RestClient.java?rev=1027164&r1=1027163&r2=1027164&view=diff
==============================================================================
--- incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/RestClient.java
(original)
+++ incubator/wink/trunk/wink-client/src/main/java/org/apache/wink/client/RestClient.java
Mon Oct 25 15:17:53 2010
@@ -190,13 +190,11 @@ public class RestClient {
                 try {
                     providersRegistry.addProvider(obj);
                 } catch (Exception e) {
-                    logger.warn(Messages.getMessage("exceptionOccurredDuringSingletonProcessing",
//$NON-NLS-1$
-                                                    obj.getClass().getCanonicalName()));
-                    logger.warn(Messages.getMessage("listExceptionDuringSingletonProcessing"),
e); //$NON-NLS-1$
+                    logger.error(Messages.getMessage("exceptionOccurredDuringSingletonProcessing",
//$NON-NLS-1$
+                                                    obj.getClass().getCanonicalName()), e);
                 } catch (NoClassDefFoundError e) {
-                    logger.warn(Messages.getMessage("exceptionOccurredDuringSingletonProcessing",
//$NON-NLS-1$
-                                                    obj.getClass().getCanonicalName()));
-                    logger.warn(Messages.getMessage("listExceptionDuringSingletonProcessing"),
e); //$NON-NLS-1$
+                    logger.error(Messages.getMessage("exceptionOccurredDuringSingletonProcessing",
//$NON-NLS-1$
+                                                    obj.getClass().getCanonicalName()), e);
                 }
             } else {
                 logger.warn(Messages.getMessage("classNotAProvider", obj)); //$NON-NLS-1$
@@ -217,13 +215,11 @@ public class RestClient {
                 try {
                     providersRegistry.addProvider(obj, priority);
                 } catch (Exception e) {
-                    logger.warn(Messages.getMessage("exceptionOccurredDuringInstanceProcessing",
//$NON-NLS-1$
-                                                    obj.getClass().getCanonicalName()));
-                    logger.warn(Messages.getMessage("listExceptionDuringInstanceProcessing"),
e); //$NON-NLS-1$
+                    logger.error(Messages.getMessage("exceptionOccurredDuringInstanceProcessing",
//$NON-NLS-1$
+                                                    obj.getClass().getCanonicalName()), e);
                 } catch (NoClassDefFoundError e) {
-                    logger.warn(Messages.getMessage("exceptionOccurredDuringInstanceProcessing",
//$NON-NLS-1$
-                                                    obj.getClass().getCanonicalName()));
-                    logger.warn(Messages.getMessage("listExceptionDuringInstanceProcessing"),
e); //$NON-NLS-1$
+                    logger.error(Messages.getMessage("exceptionOccurredDuringInstanceProcessing",
//$NON-NLS-1$
+                                                    obj.getClass().getCanonicalName()), e);
                 }
             } else {
                 logger.warn(Messages.getMessage("classNotAProvider", cls)); //$NON-NLS-1$



Mime
View raw message