HR integration review request: 225260 - Early cleaning of parser cache causes creation of a new instance of javac during groovy indexing

Tomas Zezula <[email protected]> Wed, 23 Jan 2013 21:25:01 +0100
Newsgroups gmane.comp.java.netbeans.reviewers
Message-ID <[email protected]>
Description:
http://netbeans.org/bugzilla/show_bug.cgi?id=225260


Early cleaning of parser cache causes creation of a new instance of javac
during groovy indexing. The reason of early cleaning is usage of obsolete
TimedWeakReference<T> which was introduced in time the SoftReference was
problematic which is not true anymore.

Impact assessment: The scanning of big groovy project takes longer time compared to NB 7.2

Risk assessment:  replaced obsolete TimedWeakReference with a SoftReference

Reviewed by Jan Lahoda. QA verification by Tomas Zezula.

Changesets:
http://hg.netbeans.org/jet-main/rev/f4c75d0eab18

Thanks for review,
-- Tomas