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