Bug 32276 - Use set() instead of lists in group sync
Use set() instead of lists in group sync
Product: UCS
Classification: Unclassified
Component: S4 Connector
UCS 3.0
Other Linux
: P5 normal (vote)
: UCS 3.2
Assigned To: Stefan Gohmann
Felix Botner
: interim-2
Depends on:
Blocks: 31695
  Show dependency treegraph
Reported: 2013-08-21 07:06 CEST by Stefan Gohmann
Modified: 2013-11-19 06:41 CET (History)
0 users

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Note You need to log in before you can comment on or make changes to this bug.
Description Stefan Gohmann univentionstaff 2013-08-21 07:06:37 CEST
Currently the group sync functions like group_members_sync_to_ucs or group_members_sync_from_ucs use list elements for different checks. This is rather slow while testing with 30.000 member groups.

It would be much faster to use set().
Comment 1 Stefan Gohmann univentionstaff 2013-08-29 07:41:26 CEST
fixed with r43501:
* Use set() instead of lists for the group synchronisation
  (Bug #32276)

Changelog: r43504
Comment 2 Felix Botner univentionstaff 2013-09-20 14:20:49 CEST
created 1000 users and 30 groups with all users as members

OK - group synchronization still works
OK - set() is used
OK - synchronization was faster than on an UCS 3.1-1 system with the same setup
OK - Changelog
Comment 3 Stefan Gohmann univentionstaff 2013-11-19 06:41:54 CET
UCS 3.2 has been released:

If this error occurs again, please use "Clone This Bug".