Bug #16813
closed
User batch updates should avoid useless work
Added by Peter Amstutz over 4 years ago.
Updated over 4 years ago.
Release relationship:
Auto
Description
Don't update user records (or recompute permissions) if nothing has changed.
Reported that each user record being updated would trigger a full materialized permission view update. Arvados 2.1 incremental permission updates should be an improvement, but we still want to avoid triggering a redundant refresh of each individual user's permissions.
- Description updated (diff)
- Assigned To set to Tom Clegg
- Description updated (diff)
Tom Clegg wrote:
16813-avoid-noop-user-updates @ 98bd6c43fca91b76d528c8ed5b83c655c86ffe3c -- developer-run-tests: #2088
The initial load of the user list could be done in a single query instead of loading each user record individually. But that's not a bottleneck we set out to fix in this branch, so just something to consider for future optimization.
LGTM.
- Status changed from New to In Progress
- Status changed from In Progress to Resolved
Also available in: Atom
PDF