Upgrade to new version of zincutils.

Review Request #2147 — Created May 1, 2015 and submitted

benjyw
pants
5590deb...
pants-reviews
davidt, stuhood
- New version includes massive performance improvements.
- Signature of split() has changed slightly.

CI in flight: https://travis-ci.org/pantsbuild/pants/builds/60872540

Ran this on internal Foursquare builds. Observed speed improvements. Demo'd it at Demo Day. Made assembled engineers happy.

DA
  1. Ship It!
  2. 
      
ST
  1. Do you think you guys could share what fraction of your build times you were spending on analysis, before and after the change?

    1. The fraction of the build time depends on how much zinc ends up actually doing. On very trivial builds (e.g. add a line of whitespace to a file), it was probably on the order of 50% or more analysis processing. This change cut the amount of time we spend processing analysis down by more than half, so that's a pretty huge win for iteration.

      On very large builds zinc obviously dominates, but with multiple partitions the analysis split/merges still add up.

  2. 
      
BE
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 0141dec68e6b1f08a5f2a863c85a7d74407e43b7.

Loading...