From df1fc5e234a35087aca0d4dc246265a27aa36355 Mon Sep 17 00:00:00 2001 From: Tim Hunt Date: Mon, 10 Nov 2025 13:29:12 +0000 Subject: [PATCH] Peer review checklist addition. Add to performance part of peer review checklist. Inspired by the fact that https://moodle.atlassian.net/browse/MDL-80815 got integrated with seemingly no thought about performance. --- general/development/process/peer-review/index.md | 1 + 1 file changed, 1 insertion(+) diff --git a/general/development/process/peer-review/index.md b/general/development/process/peer-review/index.md index 72aa70ae1..3c3ec7d21 100644 --- a/general/development/process/peer-review/index.md +++ b/general/development/process/peer-review/index.md @@ -140,6 +140,7 @@ Clustering is when the same code is run on different computers and an end user c Ensure that: - Any filesystem, database or cache accesses are done in the most efficient way. +- If a change is in a performance critical part of the code (for example the database layer, caching system, locks, SQL query changes ...) then we expect to see some attempt to measure the performance impact of the change reported in a comment in the tracker, or even better verified in the testing instructions. (Good examples: [testing instructions of MDL-82754](https://moodle.atlassian.net/browse/MDL-82754), [this comment in MDL-86601](https://moodle.atlassian.net/browse/MDL-86601?focusedCommentId=1019696); bad example MDL-80815.) - That any code or function that appear expensive are not in critical paths. eg; They don't load on every page. - The least possible code is running to complete the task, especially looking for hidden loops. They can appear from calling functions. - Any code that runs is not specific to a single node. (eg opcache_reset()) This ensures clusters will run correctly.