Утечка памяти в ссылке на карту weakValue для синхронизированного метода

Я создаю интерфейс для одновременного выполнения методов, абстрагируясь от деталей синхронизации (При необходимости заменить на распределенную реализацию). Я создал единственную реализацию jvm, которая позволяет использовать строки в качестве мьютекса, сохраняя их на карте, чтобы гарантировать использование одной ссылки, даже если передаются строки с разными ссылками. Параллелизм, похоже, работает нормально, однако я был удивлен, увидев, что тест показал, что количество ссылок никогда не уменьшается. Я предположил, что использования WeakValues ​​() будет достаточно для предотвращения утечек памяти, но, похоже, это не так. Может ли кто-нибудь указать, что могло быть причиной этой утечки?

public class SynchronousMethodExecutorSynchronizedImpl implements ISynchronousMethodExecutor {

// mutex map to provide string references
final Map<String, String> mutexMap = new MapMaker()
    .weakValues()
    .makeComputingMap(
        new Function<String, String>() {
        @Override
        public String apply(String id) {
            return id;
        }
    });

@Override
public Object doSynchronousMethod(String domain, String id, ISynchronousMethod synchronousMethod) {
    synchronized(mutexMap.get(domain + "." + id))
    {
        return synchronousMethod.execute();
    } 
}

}

Вот тест, который терпит неудачу при самом последнем утверждении:

public class SynchronousMethodExecutorSynchronizedImplTest extends TestCase {
int counter;
SynchronousMethodExecutorSynchronizedImpl methodExecutor;

@Override
public void before() throws Exception {
    super.before();

    methodExecutor = new SynchronousMethodExecutorSynchronizedImpl();
}

@Test
public void concurrentExecute() throws InterruptedException {
    assertEquals(0, counter);

    for(int i=0; i<1000; i++)
        getConcurrentExecutorThread().start();

    // wait for threads to complete
    Thread.sleep(1000);

    assertEquals(1, methodExecutor.mutexMap.size());

    try
    { 
        final List<long[]> infiniteList = new LinkedList<long[]>(); 

       for(long i = Long.MIN_VALUE; i < Long.MAX_VALUE; i++)
            infiniteList.add(new long[102400]); 

        fail("An OutOfMemoryError should be thrown");
    } 
    catch(OutOfMemoryError e)
    { 

    }

    assertEquals(2000, counter);
    assertEquals(0, methodExecutor.mutexMap.size());
}

// synchronous method
private ISynchronousMethod method = new ISynchronousMethod() {
    @Override
    public Object execute() {
        counter++;  
        return null;
    }
};

/**
 * Executes a line of code.
 * 
 * @return Thread
 */
private Thread getConcurrentExecutorThread() {
    return new Thread() {
        @Override
        public void run() {
            methodExecutor.doSynchronousMethod("TEST", "1", method);
            try
            {
                Thread.sleep(500);
            }
            catch (InterruptedException e)
            {

            }
            methodExecutor.doSynchronousMethod("TEST", new String("1"), method);        
        }

    };
}

}

Это последнее утверждение ломает тест: assertEquals (0, methodExecutor.mutexMap.size ());

5
задан Anthony DePalma 21 June 2011 в 01:14
поделиться