java - 异常 java.lang.OutOfMemoryError : GC overhead limit exceeded with tree map

Question

从 Hibernate 查询加载行时，我一直收到 java.lang.OutOfMemoryError: GC overhead limit exceeded。

我试过几次增加内存，但它一直在发生。我在日志中注意到它似乎指向我正在使用 TreeMap 的方法。我想知道我是否使用不当导致内存不足问题。

public List<Item> getProducts() {
    List<ProductListing> productListings = session.createCriteria(ProductListing.class)
    .createAlias("productConfiguration", "productConfiguration")
    .add(Restrictions.eq("productConfiguration.category", category))
    .add(Restrictions.eq("active", true))
    .add(Restrictions.eq("purchased", true)).list();

    Map<String, Item> items = new TreeMap<>();

    productListings.stream().forEach((productListing) -> {
        Item item = productListing.getItem();
        items.put(item.getName(), item);
    });

    return new ArrayList<>(items.values());
}

• 将值传递到 arraylist 是否安全？

• 我需要设置数组列表的大小吗？

我只是想知道我是否做错了什么。它看起来是正确的，但内存异常另有说明。

Answer 1

我称之为“加载世界”——名称 getProducts()(没有参数)有点代码味道——因为你没有限制结果集的大小，并且对于我们所知道的一切你的 Item 对象可能很大，有很多急切加载的依赖项(更不用说堆上的所有后备 Hibernate 对象)。

另一个大问题是，您将脱水实体添加到 TreeMap 中的成本很高，调用 hashCode() 并可能调用 equals() , 只是为了丢弃键并将值复制到新分配的 ArrayList 中。

抛开 ArrayList 缺少预调整大小(正确，这并不理想，虽然它应该只是慢)，为什么要使用 TreeMap 阶段？如果需要进行聚合，为什么不让数据库更有效地执行此操作(例如，通过 GROUP BY name)并使用索引，而不是将其全部拉入 map 中重新处理速度较慢？至少，通过只返回唯一的 Item，您可以跳过 map 阶段并直接复制到您的列表中(根据您的需要，甚至还有更轻量级的可能性)。

我强烈建议使用合适的 Profiler 进行测试。具体来说，它可以帮助您确定预加载对堆大小的影响。可能只是关闭它可能会使问题更易于管理。

但是，您还需要考虑代码的客户:谁真正需要所有那些Item？很可能没有人。