找出在 c# 中 sql server 连接处于打开状态的位置

Question

我继承了大量 C# dotnet 代码，这些代码显然在某些地方没有关闭与 sql server 的连接或其他东西，我得到了

The timeout period elapsed prior to obtaining a connection from the pool: This may have occurred because all pooled connections were in use and max pool size was reached

这种情况经常发生，而且通常同时发生，但我不知道有什么安排好的。手动搜索代码中的每一个小地方也是不可能的。任何想法如何在不手动检查代码中的每个函数的情况下解决这个问题？

Answer 1

不想浏览代码的问题在于，它很可能不是单个实例或区域，并且可能取决于您在生产环境中获得最多流量的位置。您可以有一个看起来相当无害的查询，可以通过.AsNoTracking() 或更好的连接/包含而不是优化不佳的单个大查询来稍微改进。也有整个设计存在缺陷的情况（我曾经进入一个项目，其中 sql 连接附加到单个类，因此没有真正的方法将连接包装在 using 语句中。我们不得不重新构建它.

如果您使用自定义事务或 sql 连接请求，请确保使用 using 和 try catch finally 与 dispose 进行包装。依赖垃圾收集器并不总是很好，因为不能保证垃圾收集器实际上会在 using 语句的末尾立即出现 dispose。

using (var conn = new SqlConnection())
{
    try
    {
        // Transactions, SQL Commands, etc
    }
    catch (SqlException ex)
    {
        // Trans rollback, error handling
    }
    finally
    {
        // Tell the garbage collector to clean up this resource now.
        conn.Dispose();
    }
}

如果您严格使用数据库上下文，则可以通过使用.AsNoTracking() 对某些语句进行微调，以用于仅检索结果集但无意修改该查询的结果的情况。类似于以下内容：

var tier = _dbContext.Tier.AsNoTracking()
    .Include(t => t.NameSchema)
    .Include(t => t.TierPackageGroups)
    .Where(t => t.TierNumber == tierNumber).FirstOrDefault();

确保尽可能使用连接或.Include，而不是延迟加载。

这不是您想听到的答案，而且我一直站在您的立场上，但很可能最好的解决方案是通过代码。你总是可以从清理几个区域开始，测量影响，再增加几个区域，测量影响等。这种蠕变调查可以帮助缩小你的搜索范围，因为你在某些方面看到了改进，或者在其他方面没有看到一些改进。

如果您确信它必须是您想要查找的单个查询，那么可以使用评估 sql 性能的工具，但我通常会请 DBA 来处理它，所以我不熟悉这些工具。如果您使用的是 Azure，则有一个面板，您可以在其中查看查询并按性能影响对它们进行组织。它不会将您指向代码，但它可以给您一个提示。

由于该问题专门针对不需要搜索代码的解决方案，因此另一个可行的答案是仅在问题上投入更多硬件，但这是一个糟糕的权宜之计，可能代价高昂且最终只是延迟问题或仅在 99% 的时间内解决问题（例如，直到您遇到交通高峰期）。我们几年前的一位客户可能已经花费了超过 50,000 美元的额外托管费用，而他们本可以让我们在最初提出请求时以不到 5,000 美元的价格解决问题。