同步方法中ConcurrentHashMap的并发级别

Question

我正在尝试解决内存泄漏问题。 堆转储分析表明， ConcurrentHashMap占据了大约98% of heap memory 。 检查了代码，结果发现ConcurrentHashMap实例化使用的是没有参数的构造函数。 concurrencyLevel的默认配置为16。在此映射实例化之后，我看到一个同步方法调用，其中将数据放入映射中。

我想知道，由于仅将数据放入同步方法中，因此将ConcurrentHashMap concurrencyLevel设置为1是否安全？

以下是示例代码片段：

private volatile Map<String, Integer> storeCache;

public void someMethod() {
    storeCache = new ConcurrentHashMap<String, Integer>();
    syncMethod();
}

private synchronized void syncMethod() {
    storeCache.put("Test", 1);
}

Answer 1

我想知道，由于仅将数据放入同步方法中，因此将ConcurrentHashMap的concurrencyLevel设置为1是否安全？

从某种意义上讲，这不会造成地图损坏，从某种意义上讲，它是安全的。 但是，它不能解决您的内存泄漏问题。 实际上，您可能不想同步对ConcurrentHashMap的访问，该访问已保证可以安全地从多个线程进行读写。 从外部进行同步将对CHM进行单线程访问，这将消除CHM与HashMap相比的许多优点。 如果删除synchronized并指定等于并发写入的估计数量的concurrencyLevel ，则可能会获得更好的性能。

至于您的内存泄漏，CHM中的键和值是强引用，这意味着Java垃圾收集器不会收集它们，即使它们在代码中的其他任何地方都不再被引用。 因此，如果您将CHM用作临时值的缓存，则当您的应用程序不再需要它们时，需要对它们进行.remove() 。

（如果您想要没有强键的ConcurrentMap的语义，则不能立即使用，但Guava提供了一个很好的选择 。）

您可能还需要检查将.put()放入映射中的键是否已正确实现.equals()和.hashCode() 。