Discussion re batching dep inference: <https://git...
# development
h
Discussion re batching dep inference: https://github.com/pantsbuild/pants/discussions/17477
b
I have a stale branch from a few months back that implements batched dependency requesting. It's just missing the changes to Python rules to support multi-file (not hard, just didn't do it) I'll push to my fork and send you the link tomorrow
h
Ooooh! Let’s discuss at link above?
b
Yeah was gonna post link there once I push to my fork 😛
I don't have bandwidth to bring it up to date or add the python implementation. I would recommend implementating the python rule code and measuring timings to see if it's worth bring it up to
main
h
That’s the thing I want to discuss on the discussion - because dep inference is inherently iterative, it’s not clear to me that batched dependency requesting is a big enough win
b
🤔
How else would we be able to batch the inference? The core code issues a request per file
*per target
The branch batches each iteration of traversal of depth
h
Let’s do the discussion on the discussion? 🙂