[hg] main-silver: #225260:Early cleaning of parser cache causes ...

  • From: Tomas Zezula < >
  • To:
  • Subject: [hg] main-silver: #225260:Early cleaning of parser cache causes ...
  • Date: Wed, 23 Jan 2013 14:37:54 -0800

changeset f4c75d0eab18 in main-silver ((none))
details: http://hg.netbeans.org/main-silver/rev/f4c75d0eab18
description:
        #225260:Early cleaning of parser cache causes creation of a new 
instance of javac during groovy indexing

diffstat:

 parsing.api/src/org/netbeans/modules/parsing/api/ParserManager.java          
     |    4 +-
 parsing.api/src/org/netbeans/modules/parsing/api/TimedWeakReference.java     
     |  129 ----------
 
parsing.api/test/unit/src/org/netbeans/modules/parsing/api/ParserManagerTest.java
 |   12 +-
 3 files changed, 11 insertions(+), 134 deletions(-)

diffs (199 lines):

diff --git 
a/parsing.api/src/org/netbeans/modules/parsing/api/ParserManager.java 
b/parsing.api/src/org/netbeans/modules/parsing/api/ParserManager.java
--- a/parsing.api/src/org/netbeans/modules/parsing/api/ParserManager.java
+++ b/parsing.api/src/org/netbeans/modules/parsing/api/ParserManager.java
@@ -43,6 +43,8 @@
 package org.netbeans.modules.parsing.api;
 
 import java.lang.ref.Reference;
+import java.lang.ref.ReferenceQueue;
+import java.lang.ref.SoftReference;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
@@ -400,7 +402,7 @@
                 throw new IllegalArgumentException("No parser for mime type: 
" + mimeType);
             }
             p = 
parserFactory.createParser(Collections.<Snapshot>emptyList());
-            cachedParsers.put(mimeType, new TimedWeakReference<Parser>(p));
+            cachedParsers.put(mimeType, new SoftReference<Parser>(p));
         }
         return p;
     }
diff --git 
a/parsing.api/src/org/netbeans/modules/parsing/api/TimedWeakReference.java 
b/parsing.api/src/org/netbeans/modules/parsing/api/TimedWeakReference.java
deleted file mode 100644
--- a/parsing.api/src/org/netbeans/modules/parsing/api/TimedWeakReference.java
+++ /dev/null
@@ -1,129 +0,0 @@
-/*
- * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
- *
- * Copyright 1997-2010 Oracle and/or its affiliates. All rights reserved.
- *
- * Oracle and Java are registered trademarks of Oracle and/or its affiliates.
- * Other names may be trademarks of their respective owners.
- *
- * The contents of this file are subject to the terms of either the GNU
- * General Public License Version 2 only ("GPL") or the Common
- * Development and Distribution License("CDDL") (collectively, the
- * "License"). You may not use this file except in compliance with the
- * License. You can obtain a copy of the License at
- * http://www.netbeans.org/cddl-gplv2.html
- * or nbbuild/licenses/CDDL-GPL-2-CP. See the License for the
- * specific language governing permissions and limitations under the
- * License.  When distributing the software, include this License Header
- * Notice in each file and include the License file at
- * nbbuild/licenses/CDDL-GPL-2-CP.  Oracle designates this
- * particular file as subject to the "Classpath" exception as provided
- * by Oracle in the GPL Version 2 section of the License file that
- * accompanied this code. If applicable, add the following below the
- * License Header, with the fields enclosed by brackets [] replaced by
- * your own identifying information:
- * "Portions Copyrighted [year] [name of copyright owner]"
- *
- * Contributor(s):
- *
- * The Original Software is NetBeans. The Initial Developer of the Original
- * Software is Sun Microsystems, Inc. Portions Copyright 1997-2006 Sun
- * Microsystems, Inc. All Rights Reserved.
- *
- * If you wish your version of this file to be governed by only the CDDL
- * or only the GPL Version 2, indicate your decision by adding
- * "[Contributor] elects to include this software in this distribution
- * under the [CDDL or GPL Version 2] license." If you do not indicate a
- * single choice of license, a recipient has the option to distribute
- * your version of this file under either the CDDL, the GPL Version 2 or
- * to extend the choice of license to its licensees as provided above.
- * However, if you add GPL Version 2 code and therefore, elected the GPL
- * Version 2 license, then the option applies only if the new code is
- * made subject to such option by the copyright holder.
- */
-
-package org.netbeans.modules.parsing.api;
-
-// XXX COPIED from org.openide.util w/ changes:
-//     weak -> soft
-//     timeout
-//     removed map key functionality
-
-import java.lang.ref.WeakReference;
-import org.openide.util.RequestProcessor;
-import org.openide.util.Utilities;
-
-/**
- * A weak reference which is held strongly for a while after last access.
- * Lifecycle:
- * <ol>
- * <li>Created. Referent held strongly. A task is scheduled into the request
- *     processor for some time in the future (currently 15 seconds).</li>
- * <li>Expired. After the timeout, the reference switches to a normal weak
- *     reference.</li>
- * <li>Touched. If the value is accessed before it is garbage collected,
- *     whether the reference is expired or not, the reference is "touched".
- *     This means that the referent is again held strongly and the timeout
- *     is started from scratch.</li>
- * <li>Dead. If after expiry there is no access before the next full GC 
cycle,
- *     the GC algorithm may reclaim the reference. In this case the reference
- *     of course dies.</li>
- * </ol>
- * @author Jesse Glick
- */
-final class TimedWeakReference<T> extends WeakReference<T> implements 
Runnable {
-
-    public static int TIMEOUT = 15000;
-
-    private static final RequestProcessor RP = new 
RequestProcessor("TimedWeakReference"); // NOI18N
-
-    private RequestProcessor.Task task;
-
-    private T o;
-
-    /** Time when the object was last time touched */
-    private long touched;
-
-    /**
-     * Create a weak reference with timeout.
-     * @param o the referent
-     */
-    public TimedWeakReference(T o) {
-        super(o, Utilities.activeReferenceQueue());
-        this.o = o;
-        task = RP.create(this);
-        task.schedule(TIMEOUT);
-    }
-
-    public synchronized void run() {
-        if (o != null) {
-            //System.err.println("Expire " + k);
-            // how long we've really been idle
-            long unused  = System.currentTimeMillis() - touched;
-            if (unused > TIMEOUT / 2) {
-                o = null;
-                touched = 0;
-            } else {
-                task.schedule(TIMEOUT - (int) unused);
-            }
-        }
-    }
-
-    public synchronized T get() {
-        if (o == null) {
-            o = super.get();
-        }
-        if (o != null) {
-            // touch me
-            //System.err.println("Touch " + k);
-            if (touched == 0) {
-                task.schedule(TIMEOUT);
-            }
-            touched = System.currentTimeMillis();
-            return o;
-        } else {
-            return null;
-        }
-    }
-
-}
diff --git 
a/parsing.api/test/unit/src/org/netbeans/modules/parsing/api/ParserManagerTest.java
 
b/parsing.api/test/unit/src/org/netbeans/modules/parsing/api/ParserManagerTest.java
--- 
a/parsing.api/test/unit/src/org/netbeans/modules/parsing/api/ParserManagerTest.java
+++ 
b/parsing.api/test/unit/src/org/netbeans/modules/parsing/api/ParserManagerTest.java
@@ -59,7 +59,6 @@
 import org.netbeans.junit.NbTestCase;
 import org.netbeans.junit.RandomlyFails;
 import org.netbeans.modules.parsing.impl.Utilities;
-import org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater;
 import 
org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater.IndexingState;
 import 
org.netbeans.modules.parsing.impl.indexing.RepositoryUpdaterTestSupport;
 import org.netbeans.modules.parsing.spi.ParseException;
@@ -97,7 +96,6 @@
     }
 
     public void testParseCache () throws Exception {
-        TimedWeakReference.TIMEOUT = 5000;
         final boolean[] called = new boolean[] {false};
         FooParser.getResultCount = 0;
         FooParser.parseCount = 0;
@@ -123,8 +121,15 @@
         assertEquals(1, FooParserFactory.createParserCount);
         assertEquals(2, FooParser.parseCount);
         assertEquals(2, FooParser.getResultCount);
-        Thread.sleep(2 * TimedWeakReference.TIMEOUT);
+
+        try {
+            byte[] dummy = new byte[(int) Runtime.getRuntime().maxMemory()];
+        } catch (Throwable e) {
+            // Ignore OME
+        }
         System.gc(); System.gc();
+
+
         called[0] = false;
         ParserManager.parse("text/foo", new UserTask() {
             @Override
@@ -222,7 +227,6 @@
         writer.close ();
         Source source = Source.create (testFile);
 
-        TimedWeakReference.TIMEOUT = 5000;
         ParserManager.parse (Collections.singleton(source), new UserTask () {
             @Override
             public void run(ResultIterator resultIterator) throws Exception {

[hg] main-silver: #225260:Early cleaning of parser cache causes ...

Tomas Zezula 01/23/2013

Project Features

About this Project

Editor was started in November 2009, is owned by Martin Ryzl, and has 147 members.
By use of this website, you agree to the NetBeans Policies and Terms of Use (revision 20140418.2d69abc). © 2013, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo
 
 
Close
loading
Please Confirm
Close