geronimo-scm mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From jdil...@apache.org
Subject svn commit: r705706 - in /geronimo/gshell/trunk: gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/ gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/ gshell-wisdom/gshell-wisdom-c...
Date Fri, 17 Oct 2008 18:55:08 GMT
Author: jdillon
Date: Fri Oct 17 11:55:08 2008
New Revision: 705706

URL: http://svn.apache.org/viewvc?rev=705706&view=rev
Log:
More logging updates

Modified:
    geronimo/gshell/trunk/gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/MetaDataRegistryImpl.java
    geronimo/gshell/trunk/gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/XStoreRecordImpl.java
    geronimo/gshell/trunk/gshell-wisdom/gshell-wisdom-core/src/main/java/org/apache/geronimo/gshell/wisdom/shell/HistoryImpl.java

Modified: geronimo/gshell/trunk/gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/MetaDataRegistryImpl.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/MetaDataRegistryImpl.java?rev=705706&r1=705705&r2=705706&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/MetaDataRegistryImpl.java
(original)
+++ geronimo/gshell/trunk/gshell-support/gshell-vfs-meta/src/main/java/org/apache/geronimo/gshell/vfs/provider/meta/data/MetaDataRegistryImpl.java
Fri Oct 17 11:55:08 2008
@@ -89,7 +89,8 @@
                 parent = lookupData(parentName);
             }
             else {
-                log.debug("Building parent tree: {}", parentName);
+                log.trace("Building parent tree: {}", parentName);
+                
                 parent = new MetaData(parentName, FileType.FOLDER);
                 registerData(parentName, parent);
             }

Modified: geronimo/gshell/trunk/gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/XStoreRecordImpl.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/XStoreRecordImpl.java?rev=705706&r1=705705&r2=705706&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/XStoreRecordImpl.java
(original)
+++ geronimo/gshell/trunk/gshell-support/gshell-xstore/src/main/java/org/apache/geronimo/gshell/xstore/XStoreRecordImpl.java
Fri Oct 17 11:55:08 2008
@@ -71,7 +71,7 @@
     public void set(final Object value) {
         assert value != null;
 
-        log.debug("Setting: {} -> {}", file.getName(), value);
+        log.trace("Setting: {} -> {}", file.getName(), value);
 
         BufferedOutputStream output = null;
         try {
@@ -96,7 +96,7 @@
     public <T> T get(final Class<T> type) {
         assert type != null;
 
-        log.debug("Getting: {}", file);
+        log.trace("Getting: {}", file);
 
         BufferedInputStream input = null;
 
@@ -110,7 +110,7 @@
 
             input = new BufferedInputStream(file.getContent().getInputStream());
             T value = marshaller.unmarshal(input);
-            log.debug("Value: {}", value);
+            log.trace("Value: {}", value);
             return value;
         }
         catch (FileSystemException e) {
@@ -127,7 +127,7 @@
     }
 
     public void close() {
-        log.debug("Closing: {}", file);
+        log.trace("Closing: {}", file);
         
         try {
             file.close();
@@ -138,7 +138,7 @@
     }
 
     public boolean delete() {
-        log.debug("Deleting: {}", file);
+        log.trace("Deleting: {}", file);
 
         try {
             return file.delete();
@@ -149,7 +149,7 @@
     }
 
     public void refresh() {
-        log.debug("Refreshing: {}", file);
+        log.trace("Refreshing: {}", file);
 
         try {
             file.refresh();

Modified: geronimo/gshell/trunk/gshell-wisdom/gshell-wisdom-core/src/main/java/org/apache/geronimo/gshell/wisdom/shell/HistoryImpl.java
URL: http://svn.apache.org/viewvc/geronimo/gshell/trunk/gshell-wisdom/gshell-wisdom-core/src/main/java/org/apache/geronimo/gshell/wisdom/shell/HistoryImpl.java?rev=705706&r1=705705&r2=705706&view=diff
==============================================================================
--- geronimo/gshell/trunk/gshell-wisdom/gshell-wisdom-core/src/main/java/org/apache/geronimo/gshell/wisdom/shell/HistoryImpl.java
(original)
+++ geronimo/gshell/trunk/gshell-wisdom/gshell-wisdom-core/src/main/java/org/apache/geronimo/gshell/wisdom/shell/HistoryImpl.java
Fri Oct 17 11:55:08 2008
@@ -46,7 +46,6 @@
     public void init() throws Exception {
         assert application != null;
         File file = application.getModel().getBranding().getHistoryFile();
-        log.debug("History file: {}", file);
         setHistoryFile(file);
     }
     
@@ -56,16 +55,11 @@
         File dir = file.getParentFile();
 
         if (!dir.exists()) {
-            boolean result = dir.mkdirs();
-            if (!result) {
-                log.warn("Failed to create base directory for history file: {}", dir);
-            }
-            else {
-                log.debug("Created base directory for history file: {}", dir);
-            }
+            // noinspection ResultOfMethodCallIgnored
+            dir.mkdirs();
         }
 
-        log.debug("Using history file: {}", file);
+        log.debug("History file: {}", file);
 
         super.setHistoryFile(file);
     }



Mime
View raw message