Performance fix for ./pants depmap --minimal

Review Request #2896 - Created Sept. 27, 2015 and submitted

Information
Eric Ayers
pants
zundel/speedup-depmap-minimal
1892, 2277
9c91139...
Reviewers
pants-reviews
gmalmquist, jsirois, nhoward_tw

Don't just shortcut the output, but skip re-traversing subtrees with depmap-minimal

Ran the following command in our repo:

time ./pants depmap <big target> --depmap-minimal

Before patch:

real    15m46.780s
user    15m24.622s
sys 0m34.392s

after patch:

real    0m16.262s
user    0m10.618s
sys 0m4.612s

No difference in output.

Garrett Malmquist
Nick Howard (Twitter)
Eric Ayers
Review request changed

Status: Closed (submitted)

Change Summary:

Commmit a92d75d42839d98d4c53fdb2aa940303d24b39f4

Loading...