Optimization question: does `_assetGraph.computeOutputs()` need to be fully computed?

[gmpassos]

I'm looking at _assetGraph.computeOutputs, and I see it's only used in this line:

build/build_runner/lib/src/io/build_output_reader.dart

Line 203 in 3b47ff2

(_assetGraph.computeOutputs()[node.id] ?? <AssetId>{}).any(

Does computeOutputs() need to be run for all nodes?

Do you think populating only the accessed nodes would be faster?

Best regards.

[davidmorgan]

It's reversing the outputs graph, so I think you have to process the whole thing in one go, then the result is cached for later lookups.

It's used if serving the results or if copying to a merged output folder ... I'm not sure if there is a use case where the performance matters too much, but I haven't actually tried to measure.