On Tue, Sep 18, 2018 at 06:39:28PM +0000, Daniel Shahaf wrote:
> Slurping the file to a string object is an antipattern.  Instead of
> using f.read() to create a 4.5GB string, it would be better to use
> json.load(f) to read the file incrementally.  That should raise an
> exception rather quickly.

That may be even better! Expect MR in a moment. Should I include some
magic text in commit message to link it with this bug?

-- 
Best Regards,
Marek Marczykowski-Górecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Attachment: signature.asc
Description: PGP signature

Reply via email to