0033625: (Order|Invoice|Cahsup)ImportEntryProcessor countEntries has bad performance with many rows in c_import_entry

diff -r ee55c9b0c2e9 src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java	Wed Jun 22 17:45:39 2016 +0200
+++ b/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java	Thu Aug 04 14:18:25 2016 +0200
@@ -95,17 +95,19 @@
     private int countEntries(String importStatus, ImportEntry importEntry) {
       final String whereClause = ImportEntry.PROPERTY_IMPORTSTATUS + "='" + importStatus
           + "' and (" + ImportEntry.PROPERTY_TYPEOFDATA + "='Order' or "
+          + ImportEntry.PROPERTY_TYPEOFDATA + "='OBPOS_Invoice'  or "
           + ImportEntry.PROPERTY_TYPEOFDATA + "='FIN_Finacc_Transaction'  or "
           + ImportEntry.PROPERTY_TYPEOFDATA + "='OBPOS_App_Cashup') and "
           + ImportEntry.PROPERTY_CREATIONDATE + "<:creationDate and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
+      qry.setMaxResults(1);
 
-      return ((Number) qry.uniqueResult()).intValue();
+      return qry.list().size();
     }
   }
 
diff -r ee55c9b0c2e9 src/org/openbravo/retail/posterminal/importprocess/InvoiceImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/InvoiceImportEntryProcessor.java	Wed Jun 22 17:45:39 2016 +0200
+++ b/src/org/openbravo/retail/posterminal/importprocess/InvoiceImportEntryProcessor.java	Thu Aug 04 14:18:25 2016 +0200
@@ -80,11 +80,12 @@
           + ImportEntry.PROPERTY_TYPEOFDATA + "='Order' and " + ImportEntry.PROPERTY_CREATIONDATE
           + "<:creationDate and " + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
+      qry.setMaxResults(1);
 
-      return ((Number) qry.uniqueResult()).intValue();
+      return qry.list().size();
     }
   }
 
diff -r ee55c9b0c2e9 src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java	Wed Jun 22 17:45:39 2016 +0200
+++ b/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java	Thu Aug 04 14:18:25 2016 +0200
@@ -83,11 +83,12 @@
           + ImportEntry.PROPERTY_CREATIONDATE + "<:creationDate and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
+      qry.setMaxResults(1);
 
-      return ((Number) qry.uniqueResult()).intValue();
+      return qry.list().size();
     }
   }
 

diff --git a/src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java
@@ -92,13 +92,13 @@
           + ImportEntry.PROPERTY_CREATEDTIMESTAMP + "<:createdtimestamp and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
       qry.setParameter("createdtimestamp", importEntry.getCreatedtimestamp());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
 
   }
diff --git a/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
@@ -100,13 +100,13 @@
           + ImportEntry.PROPERTY_CREATEDTIMESTAMP + "<:createdtimestamp and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
       qry.setParameter("createdtimestamp", importEntry.getCreatedtimestamp());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
   }
 
diff --git a/src/org/openbravo/retail/posterminal/importprocess/CustomerAddrImportEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/CustomerAddrImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/CustomerAddrImportEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/CustomerAddrImportEntryProcessor.java
@@ -81,11 +81,11 @@
           + ImportEntry.PROPERTY_CREATIONDATE + "<:creationDate and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
   }
 
diff --git a/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
@@ -83,13 +83,13 @@
           + ImportEntry.PROPERTY_CREATEDTIMESTAMP + "<:createdtimestamp and "
           + ImportEntry.PROPERTY_OBPOSPOSTERMINAL + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
       qry.setParameter("createdtimestamp", importEntry.getCreatedtimestamp());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
   }
 
diff --git a/src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java
@@ -93,12 +93,12 @@
           + "<:creationDate and " + ImportEntry.PROPERTY_OBPOSPOSTERMINAL
           + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
   }
 
diff --git a/src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java b/src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java
--- a/src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java
+++ b/src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java
@@ -94,13 +94,13 @@
           + "<:createdtimestamp and " + ImportEntry.PROPERTY_OBPOSPOSTERMINAL
           + "=:terminal and id!=:id";
       final Query qry = OBDal.getInstance().getSession()
-          .createQuery("select count(*) from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
+          .createQuery("select 1 from " + ImportEntry.ENTITY_NAME + " where " + whereClause);
       qry.setParameter("id", importEntry.getId());
       qry.setTimestamp("creationDate", importEntry.getCreationDate());
       qry.setParameter("terminal", importEntry.getOBPOSPOSTerminal());
       qry.setParameter("createdtimestamp", importEntry.getCreatedtimestamp());
-
-      return ((Number) qry.uniqueResult()).intValue();
+      qry.setMaxResults(1);
+      return qry.list().size();
     }
   }