Modified trace messages, replaced excess stacks with just message

tmobile · Aug 19, 2019 · a8c339d · a8c339d
1 parent e680688
commit a8c339d
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/...-driver-tests/src/main/java/com/tmobile/opensource/casquatch/CasquatchTestDaoBuilder.java b/...-driver-tests/src/main/java/com/tmobile/opensource/casquatch/CasquatchTestDaoBuilder.java
@@ -63,13 +63,13 @@ public CasquatchTestDaoBuilder withDDL(String ddl) {
             session.close();
         }
         catch(com.datastax.oss.driver.api.core.servererrors.AlreadyExistsException e) {
-            log.warn("DDL Exception",e);
+            log.warn("DDL Exception: {}",e.getMessage());
         }
         catch(com.datastax.oss.driver.api.core.servererrors.InvalidQueryException e) {
-            log.warn("DDL Exception",e);
+            log.warn("DDL Exception: {}",e.getMessage());
         }
         catch(Exception e) {
-            log.error("DDL Exception",e);
+            log.warn("DDL Exception: {}",e.getMessage());
             throw(e);
         }
         return this;

diff --git a/casquatch-driver/src/main/java/com/tmobile/opensource/casquatch/AbstractCasquatchObject.java b/casquatch-driver/src/main/java/com/tmobile/opensource/casquatch/AbstractCasquatchObject.java
@@ -51,7 +51,7 @@ else if(!this.getClass().equals(obj.getClass())) {
                         !method.isAnnotationPresent(com.tmobile.opensource.casquatch.annotation.CasquatchIgnore.class)) {
                     try {
                         if (!(method.invoke(this) == null && method.invoke(obj)==null) && !method.invoke(obj).equals(method.invoke(this))) {
-                            log.trace("No match on " + method.getName());
+                            log.trace("No match on {}",method.getName());
                             return false;
                         }
 

diff --git a/...uatch-driver/src/main/java/com/tmobile/opensource/casquatch/AbstractStatementFactory.java b/...uatch-driver/src/main/java/com/tmobile/opensource/casquatch/AbstractStatementFactory.java
@@ -140,7 +140,6 @@ protected Boolean allowQueryByType(Class clazz) {
         else {
             return true;
         }
-
     }
 
     /**
@@ -161,7 +160,7 @@ protected <T> BoundStatementBuilder bindIfMarked(BoundStatementBuilder boundStat
             }
         }
         catch (java.lang.IllegalArgumentException e) {
-            log.trace("Failed to bind {}",field,e);
+            log.trace("Failed to bind {}: {}",field,e.getMessage());
             return boundStatementBuilder;
         }
     }

diff --git a/casquatch-driver/src/main/java/com/tmobile/opensource/casquatch/CasquatchDao.java b/casquatch-driver/src/main/java/com/tmobile/opensource/casquatch/CasquatchDao.java
@@ -517,6 +517,7 @@ public <E extends AbstractCasquatchEntity> List<E> getAllBySolr(Class<E> c, E o,
 
     /**
      * Get all objects by passing a partially populated object. Non-Key columns are allowed.
+     * See https://docs.datastax.com/en/dse/6.0/cql/cql/cql_using/search_index/siQuerySyntax.html for details and limitations including some fields which will not be populated.
      * @param <E> Entity Object for results
      * @param c Class of object
      * @param o partially populated object