nice work on

Question

< hundreds father 404> nice work on i expect that `sys getsizeof` is lying to you a bit because it isn t recursive see but that measuring recursively is challenging here because things like TransitiveHydratedTarget intentionally structure share and would lie to a naive recursive size calculation `objgraph` might help there

witty-crayon-22786 · Answer

having said that it s pretty clear that eliminating all cases where we hold FileContent in memory seems like the big one

witty-crayon-22786 · Answer

stdout stderr are likely to be problematic as well we discussed making those lazy on `ExecuteProcessResult` initially but went for eager in the end

witty-crayon-22786 · Answer

some of this might boil down to the rust python boundary in that when an `intrinsic builtin` rule lifts a large type to be fed to a python ` rule` it should avoid actually persisting that in the `Graph`

witty-crayon-22786 · Answer

from the rust perspective things get persisted in the graph either by being part of the NodeKey input or NodeResult output enums

witty-crayon-22786 · Answer

aaaaaand going to bed

witty-crayon-22786 · Answer

thank you for looking at this

witty-crayon-22786 · Answer

hm having said that all of the things in the ` handles` map are `Values` held inside the NodeKey NodeResult enums so that s not really very helpful

witty-crayon-22786 · Answer

but still going to bed