[jOOQ#5256] DAO.delete() should delete via UpdatableRecord.delete(), …

…not via a bulk delete
66xc1 · May 12, 2016 · 60ff1c9 · 60ff1c9
1 parent ede1ae3
commit 60ff1c9
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/jOOQ/src/main/java/org/jooq/impl/DAOImpl.java b/jOOQ/src/main/java/org/jooq/impl/DAOImpl.java
@@ -199,13 +199,16 @@ else if (objects.size() == 1) {
 
     @Override
     public /* non-final */ void delete(Collection<P> objects) {
-        List<T> ids = new ArrayList<T>();
 
-        for (P object : objects) {
-            ids.add(getId(object));
+        // Execute a batch DELETE
+        if (objects.size() > 1) {
+            using(configuration).batchDelete(records(objects, true)).execute();
         }
 
-        deleteById(ids);
+        // Execute a regular DELETE
+        else if (objects.size() == 1) {
+            records(objects, true).get(0).delete();
+        }
     }
 
     @SuppressWarnings("unchecked")