[jboss-cvs] JBossCache/src/org/jboss/cache/pojo/impl ...
Elias Ross
genman at noderunner.net
Thu Jan 25 16:54:05 EST 2007
User: genman
Date: 07/01/25 16:54:05
Modified: src/org/jboss/cache/pojo/impl ObjectGraphHandler.java
PojoCacheImpl.java
Log:
JBCACHE-952 - Replace calls to Throwable.printStackTace with logging
Revision Changes Path
1.2 +2 -3 JBossCache/src/org/jboss/cache/pojo/impl/ObjectGraphHandler.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: ObjectGraphHandler.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/pojo/impl/ObjectGraphHandler.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -b -r1.1 -r1.2
--- ObjectGraphHandler.java 13 Jan 2007 15:55:06 -0000 1.1
+++ ObjectGraphHandler.java 25 Jan 2007 21:54:05 -0000 1.2
@@ -31,7 +31,7 @@
*
* @author Ben Wang
* Date: Aug 4, 2005
- * @version $Id: ObjectGraphHandler.java,v 1.1 2007/01/13 15:55:06 bwang Exp $
+ * @version $Id: ObjectGraphHandler.java,v 1.2 2007/01/25 21:54:05 genman Exp $
*/
class ObjectGraphHandler
{
@@ -105,8 +105,7 @@
pojoInstance = internal_.getAopInstance(internalFqn);
} catch (CacheException e)
{
- e.printStackTrace();
- throw new PojoCacheException("Exception in isMultipleReferenced " + e);
+ throw new PojoCacheException("Exception in isMultipleReferenced", e);
}
// check if this is a refernce
return InternalHelper.isMultipleReferenced(pojoInstance);
1.2 +8 -15 JBossCache/src/org/jboss/cache/pojo/impl/PojoCacheImpl.java
(In the diff below, changes in quantity of whitespace are not shown.)
Index: PojoCacheImpl.java
===================================================================
RCS file: /cvsroot/jboss/JBossCache/src/org/jboss/cache/pojo/impl/PojoCacheImpl.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -b -r1.1 -r1.2
--- PojoCacheImpl.java 13 Jan 2007 15:55:06 -0000 1.1
+++ PojoCacheImpl.java 25 Jan 2007 21:54:05 -0000 1.2
@@ -42,7 +42,7 @@
* Implementation class for PojoCache interface
*
* @author Ben Wang
- * @version $Id: PojoCacheImpl.java,v 1.1 2007/01/13 15:55:06 bwang Exp $
+ * @version $Id: PojoCacheImpl.java,v 1.2 2007/01/25 21:54:05 genman Exp $
*/
public class PojoCacheImpl implements PojoCache, Observer
{
@@ -79,8 +79,7 @@
}
catch (Exception e)
{
- e.printStackTrace();
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("Failed to start " + configStr, e);
}
}
@@ -97,8 +96,7 @@
}
catch (Exception e)
{
- e.printStackTrace();
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("init " + config + " failed", e);
}
delegate_ = new PojoCacheDelegate(this, this);
@@ -124,8 +122,7 @@
}
catch (CacheException e)
{
- e.printStackTrace();
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("putObject failed " + id, e);
}
}
@@ -167,8 +164,7 @@
}
catch (CacheException e)
{
- e.printStackTrace();// TODO
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("detach " + id + " failed", e);
}
}
@@ -197,8 +193,7 @@
}
catch (CacheException e)
{
- e.printStackTrace();// TODO
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("find " + id + " failed ", e);
}
}
@@ -225,8 +220,7 @@
}
catch (CacheException e)
{
- e.printStackTrace();// TODO
- throw new PojoCacheException("PojoTreeCache: " + e);
+ throw new PojoCacheException("findAll " + id + " failed", e);
}
}
@@ -264,8 +258,7 @@
}
catch (Exception e)
{
- e.printStackTrace();
- throw new PojoCacheException("PojoCacheImpl: " + e);
+ throw new PojoCacheException("Failed starting " + e, e);
}
}
More information about the jboss-cvs-commits
mailing list