Changing runtime generic exceptions to persistence exceptions

Author: emacarron

src/main/java/org/apache/ibatis/builder/xml/XMLMapperBuilder.java

@@ -27,6 +27,7 @@
 import java.util.Properties;
 
 import org.apache.ibatis.builder.BaseBuilder;
+import org.apache.ibatis.builder.BuilderException;
 import org.apache.ibatis.builder.CacheRefResolver;
 import org.apache.ibatis.builder.IncompleteElementException;
 import org.apache.ibatis.builder.MapperBuilderAssistant;
@@ -110,7 +111,7 @@ private void configurationElement(XNode context) {
       sqlElement(context.evalNodes("/mapper/sql"));
       buildStatementFromContext(context.evalNodes("select|insert|update|delete"));
     } catch (Exception e) {
-      throw new RuntimeException("Error parsing Mapper XML. Cause: " + e, e);
+      throw new BuilderException("Error parsing Mapper XML. Cause: " + e, e);
     }
   }
 

src/main/java/org/apache/ibatis/reflection/wrapper/DefaultObjectWrapperFactory.java

@@ -16,6 +16,7 @@
 package org.apache.ibatis.reflection.wrapper;
 
 import org.apache.ibatis.reflection.MetaObject;
+import org.apache.ibatis.reflection.ReflectionException;
 
 public class DefaultObjectWrapperFactory implements ObjectWrapperFactory {
 
@@ -24,7 +25,7 @@ public boolean hasWrapperFor(Object object) {
   }
 
   public ObjectWrapper getWrapperFor(MetaObject metaObject, Object object) {
-    throw new RuntimeException("The DefaultObjectWrapperFactory should never be called to provide an ObjectWrapper.");
+    throw new ReflectionException("The DefaultObjectWrapperFactory should never be called to provide an ObjectWrapper.");
   }
 
 }

src/main/java/org/apache/ibatis/type/TypeHandlerRegistry.java

@@ -321,14 +321,14 @@ public <T> TypeHandler<T> getInstance(Class<?> javaTypeClass, Class<?> typeHandl
       } catch (NoSuchMethodException ignored) {
         // ignored
       } catch (Exception e) {
-        throw new RuntimeException("Failed invoking constructor for handler " + typeHandlerClass, e);
+        throw new TypeException("Failed invoking constructor for handler " + typeHandlerClass, e);
       }
     }
     try {
       Constructor<?> c = typeHandlerClass.getConstructor();
       return (TypeHandler<T>) c.newInstance();
     } catch (Exception e) {
-      throw new RuntimeException("Unable to find a usable constructor for " + typeHandlerClass, e);
+      throw new TypeException("Unable to find a usable constructor for " + typeHandlerClass, e);
     }
   }