commit a965d45170875a1884f522a637b24df90054b842
Author: Stu Hood <stuhood@twitter.com>
Date:   Fri Aug 11 14:23:21 2017 -0700

    Zinc 1.0.0-RC3 memory and output improvements (#4807)
    
    ### Problem
    
    While working through #4477, it became obvious that:
    1. the performance achieved when the analysis cache is missed is pretty abysmal
    2. a `log4j` JMX error was being logged during startup
    3. the `forkJava` and `javaHome` settings are redundant: the only reason to pass `javaHome` would be to trigger forking Java
    4. a large amount of unnecessary classloading was happening due to #4744
    
    ### Solution
    
    1. Set the default `zinc.analysis.cache.limit` value to `Int.MaxValue`, which will allow users who want to cap the size to set it lower, but otherwise not lead to performance cliffs when a target has more dependencies than the current limit.
    2. Implicitly disable log4j JMX usage by setting the `log4j2.disable.jmx` property if it is not already set.
    3. Remove the `forkJava` setting, to prepare to use the `javaHome` setting explicitly in #4729
    4. Enable usage of `ClassLoaderCache`
    
    ### Result
    
    Fixes #4744, and removes a few more blockers for #4729.

 src/scala/org/pantsbuild/zinc/analysis/AnalysisMap.scala | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)