Performance fix for consolidated classpath
Review Request #4184 — Created Aug. 23, 2016 and submitted — Latest diff uploaded
Information | |
---|---|
molsen | |
pants | |
3806 | |
Reviewers | |
pants-reviews | |
mateor, nhoward_tw, patricklaw, stuhood, zundel |
Previously we fetched the classpath entries for each target which was very expensive. This change fetches them upfront. Prior to change: 23.419 main:consolidate-classpath 23.414 main:consolidate-classpath:consolidate-classpath After change: 1.344 main:consolidate-classpath 1.340 main:consolidate-classpath:consolidate-classpath
CI pending: https://travis-ci.org/pantsbuild/pants/builds/154601783