Performance fix for consolidated classpath

Review Request #4184 - Created Aug. 24, 2016 and submitted

Information
Matt Olsen
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

Issues

  • 0
  • 2
  • 0
  • 2
Description From Last Updated
Stu Hood
Matt Olsen
Stu Hood
Ity Kaul
Mateo Rodriguez
Matt Olsen
Matt Olsen
Review request changed

Status: Closed (submitted)

Change Summary:

commit 46a4b43f02d9aa4558ed225750017c1d4ea20ea3

Loading...