3

我在使用以下实体框架查询时遇到性能问题：

using (MyEntities context = new MyEntities())
{
    return context.Companies
                  .Single(c => c.CompanyId == company.CompanyId)
                  .DataFile.Sum(d => d.FileSize);
}

在 SQL 探查器中进行跟踪时，我看到以下 SQL 命令：

exec sp_executesql N'SELECT 
[Extent1].[DataFileID] AS [DataFileID], 
[Extent1].[LocalFileName] AS [LocalFileName], 
[Extent1].[ServerFileName] AS [ServerFileName], 
[Extent1].[DateUploaded] AS [DateUploaded], 
[Extent1].[FileSize] AS [FileSize], 
[Extent1].[CompanyID] AS [CompanyID]
FROM [dbo].[DataFile] AS [Extent1]
WHERE [Extent1].[CompanyID] = @EntityKeyValue1',N'@EntityKeyValue1 int',@EntityKeyValue1=16

据我所知，所有数据文件行都被返回（超过 10,000 行）到内存中，然后Sum()正在发生。

编辑：

根据 Patryk 的建议，我已将查询更改为：

using (MyEntities context = new MyEntities())
{
    return context.Companies
                  .Where(c => c.CompanyId == company.CompanyId)
                  .Select(x => x.DataFiles.Sum(d => d.FileSize))
                  .Single();
}

SQL 跟踪如下所示：

SELECT TOP (2) 
(
    SELECT 
        SUM([Extent2].[FileSize]) AS [A1]
    FROM 
        [dbo].[DataFile] AS [Extent2]
    WHERE 
        [Extent1].[CompanyId] = [Extent2].[CompanyID]
) AS [C1]
FROM 
    [dbo].[Company] AS [Extent1]
WHERE 
    [Extent1].[CompanyId] = 16

这要好得多，但是，基本上我只想要这样简单快捷的东西：

SELECT SUM(FileSize) FROM DataFile WHERE CompanyId = 16
4

2 回答 2

5

首先......自从我上次检查以来，任何一个实体框架都得到了改进。所有人都认为表达式.Single(c => c.CompanyId == company.CompanyId)应该失败，因为实体框架应该在Expression.Constant<Company>. 我怀疑实际上你混淆了你的代码清单。

这有点错误的原因是由于其.Single(Expression)工作原理。与大多数 LinqIQueryable<T>扩展方法不同，它会立即进行评估。

using (MyEntities context = new MyEntities())
{
    return context.Companies
                  .Single(c => c.CompanyId == company.CompanyId)
                  .DataFile.Sum(d => d.FileSize);
}

相当于

using (MyEntities context = new MyEntities())
{
    Company company = context.Companies.Single(c => c.CompanyId == company.CompanyId);
    List<DataFile> dataFiles = company.DataFile
    return dataFiles.Sum(d => d.FileSize);
}

为你分解这个。糟糕的表现来自多个方面。

第一个是.Single()强制评估查询，返回Company（您需要，但不需要）。如果幸运的话，EF 可能很聪明，只需从缓存中提取即可。

第二行提取该公司的所有数据文件（因为List<T>其中没有任何实体框架代码。这意味着它必须拉下整个列表。

然后你知道的第三部分是.Sum(). 但是如果你检查实际的.Sum()实现，它实际上IEnumerable.Sum()与实体框架无关。签名完全不同。

与 ELinq 一起工作的IQueryable<T>.Sum<T,TValue>(Expression<Func<T,TValue>> projection)一个是 Linq to Object 一是IEnumerable<T>.Sum<T,TValue>(Func<T.TValue> projection)

TLDR：

简而言之，需要一些时间来了解 LinqToEF 的开始和结束位置。您的代码有效的唯一原因是 EF 延迟加载。但我建议您在出现性能问题时关闭 EF Lazy Loading，因为它通常会隐藏对 Linq 的不了解。

于 2013-09-26T14:08:34.300 回答
0

select 语句实际上应该只返回查询定义的内容，

select * from Extend1 where CompanyID = 16

意思是，不，它应该只返回 CompanyID = 16 的所有行。

Tbh 不知道实体框架的行为方式，但是如果您使用 nhibernate 进行查询，例如，linq 查询.First(p=>p.Id==16)将执行 Select Top(1)

也许这篇文章可以帮助您优化生成的查询：Force Entity Framework to use SQL parameterization for better SQL proc cache reuse

于 2013-09-26T11:52:42.230 回答