Challenge: stazrouti submission #143
Conversation
| const fs = require('fs'); | ||
|
|
||
| // Reading the input.txt file | ||
| fs.readFile('../../input.txt', 'utf8', (err, data) => { |
There was a problem hiding this comment.
This loads the whole file in memory I believe. It can trigger ab OOM
There was a problem hiding this comment.
@aboullaite
but the memory usage is not indicated in the requirement !!
There was a problem hiding this comment.
Its 22G of data! There is no such thing as infinite memory
There was a problem hiding this comment.
@aboullaite
but there is no need for a lot of ressources
There was a problem hiding this comment.
Loading the whole file in memory would require 22G, plus the overhead of any data structure to handle your code, and then everything else. That's a lot of memory
There was a problem hiding this comment.
@aboullaite
sorry but no need for 22 G, i try it
There was a problem hiding this comment.
I am no nodejs expert, but this article and many others point to that readFile load file in memory https://betterprogramming.pub/a-memory-friendly-way-of-reading-files-in-node-js-a45ad0cc7bb6
No description provided.