kbase / blobstore

Simple S3 backed file storage.
MIT License
0 stars 3 forks source link

Add readers to a node in one operation vs N #138

Open MrCreosote opened 2 years ago

MrCreosote commented 2 years ago

No idea why I did it this way in the first place... https://docs.mongodb.com/manual/reference/operator/update/addToSet/#-each-modifier

MrCreosote commented 2 years ago

https://docs.mongodb.com/v3.6/reference/operator/update/addToSet/#each-modifier

MrCreosote commented 2 years ago

No idea why I did it this way in the first place...

Probably because adding multiple users to a read ACL at the same time virtually never happens. IOW, this is super low priority. Adding a single reader to multiple nodes would have a much bigger performance benefit in KBase land.