From b9f35c95ac7a7dbf853fb9f58f2c7b4688fa6e1d Mon Sep 17 00:00:00 2001
From: Simon Brandhof <simon.brandhof@sonarsource.com>
Date: Tue, 31 Jul 2018 22:04:55 +0200
Subject: [PATCH] SONAR-11095 Perf hotspot when background task sends issue
 notifications

---
 .../db/permission/AuthorizationMapper.xml     | 65 ++++++++++---------
 1 file changed, 33 insertions(+), 32 deletions(-)

diff --git a/server/sonar-db-dao/src/main/resources/org/sonar/db/permission/AuthorizationMapper.xml b/server/sonar-db-dao/src/main/resources/org/sonar/db/permission/AuthorizationMapper.xml
index 3df9e766e1e..45fb21902cb 100644
--- a/server/sonar-db-dao/src/main/resources/org/sonar/db/permission/AuthorizationMapper.xml
+++ b/server/sonar-db-dao/src/main/resources/org/sonar/db/permission/AuthorizationMapper.xml
@@ -400,37 +400,38 @@
   </select>
 
   <select id="keepAuthorizedLoginsOnProject" parameterType="map" resultType="String">
-      SELECT u.login
-      FROM users u
-      INNER JOIN user_roles ur ON ur.user_id = u.id
-      INNER JOIN projects p ON p.kee = #{projectKey,jdbcType=VARCHAR}
-      WHERE
-        ur.organization_uuid = p.organization_uuid
-        AND ur.resource_id = p.id
-        AND ur.role = #{permission,jdbcType=VARCHAR}
-        AND u.login IN <foreach collection="logins" open="(" close=")" item="login" separator=",">#{login}</foreach>
-
-      UNION
-
-      SELECT u.login
-      FROM users u
-      INNER JOIN projects p ON p.kee = #{projectKey,jdbcType=VARCHAR}
-      INNER JOIN group_roles gr ON gr.organization_uuid = p.organization_uuid
-      INNER JOIN groups_users gu ON gr.group_id = gu.group_id
-      WHERE
-        gu.user_id = u.id
-        AND gr.role = #{permission,jdbcType=VARCHAR}
-        AND u.login IN <foreach collection="logins" open="(" close=")" item="login" separator=",">#{login}</foreach>
-
-    <if test="permission == 'user' or permission == 'codeviewer'">
-      UNION
-
-      SELECT u.login
-      FROM users u
-      INNER JOIN projects p ON p.kee = #{projectKey,jdbcType=VARCHAR}
-      WHERE
-        p.private = ${_false}
-        AND u.login IN <foreach collection="logins" open="(" close=")" item="login" separator=",">#{login}</foreach>
-    </if>
+    select u.login
+    from users u
+    where
+      u.login in <foreach collection="logins" open="(" close=")" item="login" separator=",">#{login,jdbcType=VARCHAR}</foreach>
+      and (
+        exists (
+          select 1
+          from user_roles ur
+          inner join projects p on p.id = ur.resource_id and p.organization_uuid = ur.organization_uuid
+          where
+            p.kee =  #{projectKey,jdbcType=VARCHAR}
+            and ur.role = #{permission,jdbcType=VARCHAR}
+            and ur.user_id = u.id
+        ) or exists (
+          select 1
+          from projects p
+          inner join group_roles gr on gr.resource_id = p.id and gr.organization_uuid = p.organization_uuid
+          inner join groups_users gu on gu.group_id = gr.group_id
+          where
+            p.kee  =  #{projectKey,jdbcType=VARCHAR}
+            and gu.user_id = u.id
+            and gr.role = #{permission,jdbcType=VARCHAR}
+        )
+        <if test="permission == 'user' or permission == 'codeviewer'">
+          or exists (
+            select 1
+            from projects p
+            where
+              p.kee =  #{projectKey,jdbcType=VARCHAR}
+              and p.private = ${_false}
+          )
+        </if>
+      )
   </select>
 </mapper>
-- 
2.39.5