Bug/800 user tag update #14
Conversation
| up(db, next) { | ||
| // update records | ||
| const users = db.collection('users'); | ||
| users.find({}).snapshot().forEach( |
There was a problem hiding this comment.
@ra-ver I don't think this will be an issue at current volume, but wanna bring it to your attention just in case.
This loads up all the user data into memory and writes a transformed version back. Just wanna make sure we're not gonna run out of memory in whatever vm we'll be running this in.
There was a problem hiding this comment.
non problem, just checked current prod persona, it's just 179 KB right now :)
There was a problem hiding this comment.
@hlminh2000 does a foreach load everything into memory? I could be reading this link incorrectly, but it sounds like a doc is loaded into memory once you operate on it. So toArray would load everything but foreach would load each one?
(https://stackoverflow.com/questions/25507866/how-can-i-use-a-cursor-foreach-in-mongodb-using-node-js)
|
Removed migrations. |
No description provided.