As of today, when building the list of users each user need a query to the database to build the PFUser object. While it's not perfect in general, it's killer in case of bind to `registred_users` group with thousands of users. This change introduce many things: - First, when registred_users is part of selected groups, there is no need to query other groups as it already means "all possible users". - Second, when the list is built at least once, it's kept in memory and re-used if there are several fields with the same configuration on the same artifact view. - Third, full users are queried so we can create fullfledged users right away instead of having to re-query everything everytime. All in all, this removes as many SQL queries as number of users on the platform. 5k users => 5k less SQL queries. Part of: request #31598 (Multi)SelectBox field bound to large list of users (5k+) performance issue Change-Id: I665e6b68655e134535b1e6c05a884d95b39765ec

+188 −28

Author

Manuel Vacelet (vaceletm) 2023-04-25 16:14

Committer

Manuel Vacelet (vaceletm) 2023-05-04 11:49

Hash 22aa0e183158ec9783f6315bd3f8f3c5daab74d7

Parent d05ba59708d412123cb69f9e9df1d8525906153b

Reference git #tuleap/stable/22aa0e183158ec9783f6315bd3f8f3c5daab74d7