code polish

Author: emacarron

src/main/java/org/apache/ibatis/executor/resultset/NestedResultSetHandler.java

@@ -164,13 +164,13 @@ private boolean applyNestedResultMappings(ResultSet rs, ResultMap resultMap, Met
               targetMetaObject.add(rowValue);
             } else {
               if (!parentIsNew && !isAncestor) { 
-                throw new ExecutorException("Trying to set a 1 to 1 child element twice  for '" + resultMapping.getProperty() + "'. Check your id properties.");
+                throw new ExecutorException("Trying to set a 1 to 1 child element twice for '" + resultMapping.getProperty() + "'. Check your id properties.");
               }
               metaObject.setValue(resultMapping.getProperty(), rowValue);
             }
             foundValues = true;
           }
-        } catch (Exception e) {
+        } catch (SQLException e) {
           throw new ExecutorException("Error getting nested result map values for '" + resultMapping.getProperty() + "'.  Cause: " + e, e);
         }
       }
@@ -243,9 +243,14 @@ private CacheKey createAbsoluteKey(ResultMap resultMap, ResultSet rs, String col
     return cacheKey;
   }
 
-  private CacheKey getCombinedKey(CacheKey rowKey, CacheKey parentRowKey) throws CloneNotSupportedException {
+  private CacheKey getCombinedKey(CacheKey rowKey, CacheKey parentRowKey) {
     if (rowKey.getUpdateCount() > 1 && parentRowKey.getUpdateCount() > 1) {
-      CacheKey combinedKey = rowKey.clone();
+      CacheKey combinedKey;
+      try {
+        combinedKey = rowKey.clone();
+      } catch (CloneNotSupportedException e) {
+        throw new ExecutorException("Error cloning cache key.  Cause: " + e, e);
+      }
       combinedKey.update(parentRowKey);
       return combinedKey;
     }