Merged HEAD-BUG-FIX (4.3/Cloud) to HEAD (4.3/Cloud)

57233: Merged V4.2-BUG-FIX (4.2.1) to HEAD-BUG-FIX (Cloud/4.3)
      57116: Merged V4.1-BUG-FIX (4.1.7) to V4.2-BUG-FIX (4.2.1)
         56986: MNT-9829: Merged V4.1.5 (4.1.5.10) to V4.1-BUG-FIX (4.1.7)
            56818: Fix for     MNT-9706  Big amount of alf_acl_change_set entries slowing down massively index times and prevent access to report / summary pages of SOLR
            56848: Fix for     MNT-9706  Big amount of alf_acl_change_set entries slowing down massively index times and prevent access to report / summary pages of SOLR
            - moved to join, count and group by    as having is causing issues.
            56868: Fix for     MNT-9706  Big amount of alf_acl_change_set entries slowing down massively index times and prevent access to report / summary pages of SOLR
            - also protect direct acl fetch with node exists test
            - fix test to use a sensible loop .....


git-svn-id: https://svn.alfresco.com/repos/alfresco-enterprise/alfresco/HEAD/root@61788 c4b6b30b-aa2e-2d43-bbcb-ca4b014f7261

config/alfresco/ibatis/org.hibernate.dialect.Dialect/solr-common-SqlMap.xml

@@ -36,15 +36,10 @@
         select
             acs.id              as id,
             acs.commit_time_ms  as commit_time_ms,
-            (select
+            count(acl_id) as acl_count
-                 count(acl.id)
-             from
-                 alf_access_control_list acl
-             where
-                 acl.acl_change_set = acs.id
-            ) as acl_count
         from
             alf_acl_change_set acs
+        join alf_access_control_list acl on (acl.acl_change_set = acs.id and exists (select 1 from alf_node admnode where admnode.acl_id = acl.id))
         <where>
            <if test="fromCommitTimeInclusive != null">
                <![CDATA[acs.commit_time_ms >= #{fromCommitTimeInclusive}]]>
@@ -59,6 +54,7 @@
                and <![CDATA[acs.id < #{toIdExclusive}]]>
            </if>
         </where>
+        group by acs.commit_time_ms, acs.id
         order by acs.commit_time_ms ASC, acs.id ASC
     </select>
 
@@ -69,8 +65,9 @@
         from
             alf_access_control_list acl
         <where>
+            exists (select 1 from alf_node admnode where admnode.acl_id = acl.id)
             <if test="fromIdInclusive != null">
-               <![CDATA[acl.id >= #{fromIdInclusive}]]>
+               and <![CDATA[acl.id >= #{fromIdInclusive}]]>
             </if>
             <if test="ids != null">
                 and acl.acl_change_set in 
@@ -84,27 +81,14 @@
     </select>
 
     <select id="select_Txns" parameterType="SOLRTrackingParameters" resultMap="result_Transaction">
-        select
+       select
             txn.id              as id,
             txn.commit_time_ms  as commit_time_ms,
-            (select
+            count(case when node.type_qname_id != #{deletedTypeQNameId} then 1 end) as updates,
-                 count(node.id)
+            count(case when node.type_qname_id = #{deletedTypeQNameId} then 1 end) as deletes
-             from
-                 alf_node node
-             where
-                 txn.id = node.transaction_id and
-                 node.type_qname_id != #{deletedTypeQNameId}
-            ) as updates,
-            (select
-                 count(node.id)
-             from
-                 alf_node node
-             where
-                 txn.id = node.transaction_id and
-                 node.type_qname_id = #{deletedTypeQNameId}
-            ) as deletes
         from
             alf_transaction txn
+        join alf_node node on (txn.id = node.transaction_id)
         <where>
            <if test="fromCommitTimeInclusive != null">
                <![CDATA[txn.commit_time_ms >= #{fromCommitTimeInclusive}]]>
@@ -119,6 +103,7 @@
                and <![CDATA[txn.id < #{toIdExclusive}]]>
            </if>
         </where>
+        group by txn.commit_time_ms, txn.id
         order by txn.commit_time_ms ASC, txn.id ASC
     </select>
 

source/test-java/org/alfresco/repo/solr/SOLRTrackingComponentTest.java

@@ -21,6 +21,7 @@ package org.alfresco.repo.solr;
 import java.io.InputStream;
 import java.io.Serializable;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collection;
 import java.util.HashMap;
 import java.util.HashSet;
@@ -138,25 +139,20 @@ public class SOLRTrackingComponentTest extends TestCase
     {
         List<AclChangeSet> cs = solrTrackingComponent.getAclChangeSets(null, null, null, null, 50);
         assertTrue("Expected results to be limited in number", cs.size() <= 50);
-        List<Long> aclChangeSetIds = new ArrayList<Long>(50);
         int totalAcls = 0;
         for (AclChangeSet aclChangeSet : cs)
         {
-            aclChangeSetIds.add(aclChangeSet.getId());
             totalAcls += aclChangeSet.getAclCount();
         }
         int totalAclsCheck = 0;
-        Long fromAclId = null;
+        
-        while (true)
+        for (AclChangeSet aclChangeSet : cs)
         {
-            List<Acl> acls = solrTrackingComponent.getAcls(aclChangeSetIds, fromAclId, 2);
+            List<Acl> acls = solrTrackingComponent.getAcls(Arrays.asList(new Long[]{aclChangeSet.getId()}), null, 200);
-            if (acls.size() == 0)
+            assertEquals(aclChangeSet.getAclCount(), acls.size());
-            {
-                break;
-            }
             totalAclsCheck += acls.size();
-            fromAclId = acls.get(acls.size() - 1).getId() + 1;
         }
+        
         // Double check number of ACLs
         assertEquals("ACL count should have matched", totalAcls, totalAclsCheck);
     }