Merge pull request #2846 from CSCfi/no-cache-reload

feat: don't bother with the cache reload in rename-user
CSCfi · Feb 15, 2022 · fed0a39 · fed0a39
2 parents bce32c9 + d6ae8b4
commit fed0a39
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -18,6 +18,7 @@ Changes since v2.24
 - Validate GA4GH claims by the public key from trusted issuer jku (instead of OIDC configuration). Configure `:ga4gh-visa-trusted-issuers` if needed. (#2798)
 - Pollers have been made more robust, including timeouts for email sending (#2841)
 - SSL certificate can now be configured and SSL enabled. HTTP port can be disabled. (#2844)
+- `rename-user` will not bother reloading the application cache anymore (since it is usually run in a separate process from the server).
 
 ## v2.24 "Heikkiläntie" 2022-01-17
 

diff --git a/src/clj/rems/db/fix_userid.clj b/src/clj/rems/db/fix_userid.clj
@@ -289,7 +289,7 @@
                   [(:name (meta f))
                    (f old-userid new-userid simulate?)]))]
     (remove-old-user old-userid simulate?)
-    (rems.db.applications/reload-cache!)
+    ;; (rems.db.applications/reload-cache!) ; can be useful if running from REPL
     result))
 
 (comment