directory-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From smckin...@apache.org
Subject directory-fortress-core git commit: FC-105 - [fortress-core] - stack trace may be lost
Date Fri, 22 May 2015 13:32:21 GMT
Repository: directory-fortress-core
Updated Branches:
  refs/heads/master 869db15ac -> 6568d40be


FC-105 - [fortress-core] - stack trace may be lost


Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/6568d40b
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/6568d40b
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/6568d40b

Branch: refs/heads/master
Commit: 6568d40bea3b965a61118fb0b5ba06acd3d8469f
Parents: 869db15
Author: Shawn McKinney <smckinney@apache.org>
Authored: Fri May 22 08:32:18 2015 -0500
Committer: Shawn McKinney <smckinney@apache.org>
Committed: Fri May 22 08:32:18 2015 -0500

----------------------------------------------------------------------
 .../apache/directory/fortress/core/cli/Options.java  |  2 +-
 .../fortress/core/ldap/ApacheDsDataProvider.java     | 15 ++++++---------
 2 files changed, 7 insertions(+), 10 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/6568d40b/src/main/java/org/apache/directory/fortress/core/cli/Options.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/cli/Options.java b/src/main/java/org/apache/directory/fortress/core/cli/Options.java
index 408302b..a08ce45 100755
--- a/src/main/java/org/apache/directory/fortress/core/cli/Options.java
+++ b/src/main/java/org/apache/directory/fortress/core/cli/Options.java
@@ -90,7 +90,7 @@ public class Options implements java.io.Serializable
     private final CmdLineParser.Option postalOfficeBox;
     private final CmdLineParser.Option protocol;
     private final CmdLineParser.Option member;
-    
+
     /**
      * @param parser
      */

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/6568d40b/src/main/java/org/apache/directory/fortress/core/ldap/ApacheDsDataProvider.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/ldap/ApacheDsDataProvider.java
b/src/main/java/org/apache/directory/fortress/core/ldap/ApacheDsDataProvider.java
index f9b9bc5..f2549c1 100644
--- a/src/main/java/org/apache/directory/fortress/core/ldap/ApacheDsDataProvider.java
+++ b/src/main/java/org/apache/directory/fortress/core/ldap/ApacheDsDataProvider.java
@@ -216,10 +216,8 @@ public abstract class ApacheDsDataProvider
             throw new CfgRuntimeException( GlobalErrIds.FT_APACHE_LDAP_POOL_INIT_FAILED,
error, ex );
         }
 
-        //PoolableObjectFactory<LdapConnection> poolFactory = new DefaultPoolableLdapConnectionFactory(
config );
         PoolableObjectFactory<LdapConnection> poolFactory = new ValidatingPoolableLdapConnectionFactory(
config );
 
-
         // Create the Admin pool
         adminPool = new LdapConnectionPool( poolFactory );
         adminPool.setTestOnBorrow( true );
@@ -270,7 +268,6 @@ public abstract class ApacheDsDataProvider
                 logPw = Config.getProperty( LDAP_LOG_POOL_PW );
             }
             logConfig.setCredentials( logPw );
-            //poolFactory = new DefaultPoolableLdapConnectionFactory( logConfig );
             poolFactory = new ValidatingPoolableLdapConnectionFactory( logConfig );
             logPool = new LdapConnectionPool( poolFactory );
             logPool.setTestOnBorrow( true );
@@ -1440,7 +1437,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new RuntimeException( e.getMessage() );
+            throw new RuntimeException( e.getMessage(), e );
         }
     }
 
@@ -1459,7 +1456,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new RuntimeException( e.getMessage() );
+            throw new RuntimeException( e.getMessage(), e );
         }
     }
 
@@ -1478,7 +1475,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new RuntimeException( e.getMessage() );
+            throw new RuntimeException( e.getMessage(), e );
         }
     }
 
@@ -1497,7 +1494,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new LdapException( e.getMessage() );
+            throw new LdapException( e.getMessage(), e );
         }
     }
 
@@ -1516,7 +1513,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new LdapException( e.getMessage() );
+            throw new LdapException( e.getMessage(), e );
         }
     }
 
@@ -1535,7 +1532,7 @@ public abstract class ApacheDsDataProvider
         }
         catch ( Exception e )
         {
-            throw new LdapException( e.getMessage() );
+            throw new LdapException( e.getMessage(), e );
         }
     }
 


Mime
View raw message