#1190 - MySql throwing DataIntegrityException when it should throw the more specific DuplicateKeyException

src/main/java/io/ebean/config/dbplatform/SqlCodeTranslator.java

@@ -34,6 +34,10 @@ public class SqlCodeTranslator implements SqlExceptionTranslator {
   public PersistenceException translate(String message, SQLException e) {
 
     DataErrorType errorType = map.get(e.getSQLState());
+    if (errorType == null) {
+      // fall back to error code
+      errorType = map.get(String.valueOf(e.getErrorCode()));
+    }
     if (errorType != null) {
       switch (errorType) {
         case AcquireLock:

src/main/java/io/ebean/config/dbplatform/mysql/MySqlPlatform.java

@@ -42,8 +42,8 @@ public class MySqlPlatform extends DatabasePlatform {
     this.exceptionTranslator =
       new SqlErrorCodes()
         .addAcquireLock("1205")
-        .addDuplicateKey("1062")
-        .addDataIntegrity("630","839","840","893","1169","1215","1216","1217","1364","1451","1452","1557","23000")
+        .addDuplicateKey("1062", "1169", "23000")
+        .addDataIntegrity("630", "839", "840", "893", "1215", "1216", "1217", "1364", "1451", "1452", "1557")
         .build();
 
     this.openQuote = "`";
@@ -79,7 +79,7 @@ public class MySqlPlatform extends DatabasePlatform {
     // NOWAIT and SKIP LOCKED currently not supported with MySQL
     return sql + " for update";
   }
-  
+
   @Override
   protected void escapeLikeCharacter(char ch, StringBuilder sb) {
     sb.append('|').append(ch);

src/test/java/org/tests/rawsql/TestRawSqlOrmQuery.java

@@ -8,6 +8,8 @@ import io.ebean.PagedList;
 import io.ebean.Query;
 import io.ebean.RawSql;
 import io.ebean.RawSqlBuilder;
+import io.ebean.annotation.IgnorePlatform;
+import io.ebean.annotation.Platform;
 import org.tests.model.basic.Customer;
 import org.tests.model.basic.Order;
 import org.tests.model.basic.ResetBasicData;
@@ -67,6 +69,7 @@ public class TestRawSqlOrmQuery extends BaseTestCase {
   }
 
   @Test
+  @IgnorePlatform(Platform.MYSQL)
   public void test_upperCaseSql() {
 
     ResetBasicData.reset();
@@ -185,14 +188,14 @@ public class TestRawSqlOrmQuery extends BaseTestCase {
     query.setRawSql(rawSql);
 
     query.setMaxRows(100);
-    
+
     if (isSqlServer()) {
       query.order("coalesce(shipDate, getdate()) desc");
       query.findList();
 
       assertThat(sqlOf(query)).contains("order by coalesce(o.ship_date, getdate()) desc, o.id");
       assertThat(sqlOf(query)).contains("select top 100");
-      
+
     } else {
       query.order("coalesce(shipDate, now()) desc");
       query.findList();