jsha / blocktogether

Share your blocks and subscribe to others'
GNU General Public License v3.0
330 stars 68 forks source link

Update-users.js looks up users twice #186

Closed jsha closed 9 years ago

jsha commented 9 years ago

Once when choosing users to update, then a second time in storeUser.

StoreUser should have a second argument, for use within update-users, that provides the user object if it was recently fetched from the DB. This should reduce the number of DB queries and the CPU usage of update-users.js.

jsha commented 9 years ago

Fixed.