为什么我的 linq to SQL 查询很慢？

Question

我得到了这个在生产环境中非常慢的查询。我想知道为什么以及是否有更好的方法来实现相同的结果。

public async Task<Membership> FindByEmailByAccessL1OrL3OrL4Async(string email)
{
    return await (from m in this.Queryable()
                  where m.Email == email
                        && (m.RoleMemberships.Select(r => r.RoleId).Contains(RoleConstants.ACCESSGRANTEDL1ID)
                            || m.RoleMemberships.Select(r => r.RoleId).Contains(RoleConstants.ACCESSGRANTEDL3ID)
                            || m.RoleMemberships.Select(r => r.RoleId).Contains(RoleConstants.ACCESSGRANTEDL4ID))
                  select m).SingleOrDefaultAsync();
}

简而言之，此查询的目的是通过电子邮件获取用户成员资格并进行登录。我得到的用户会员越多，这个查询就越慢。

谢谢，

大卫

编辑

this.Queryable() 相当于做select * from Memberships。 Membership表有很多角色，一个角色有很多成员。

此外，成员资格表最多需要 12 秒才能获取 240 行的登录信息。我必须按角色过滤，因为电子邮件值在系统中不是唯一的。只有某些角色有权登录，并且这些角色将拥有唯一的电子邮件。这就是我必须过滤的原因。此查询返回一个成员资格以继续登录，该成员资格已通过上述查询选择。所以没有IQueryable、IEnumerable 或List，只有SingleOrDefault 值。

Answer 1

我认为生成的查询非常糟糕，因为您多次执行相同的操作：

m.RoleMemberships.Select(r => r.RoleId)

相反，您可以这样做一次：

public Task<Membership> FindByEmailByAccessL1OrL3OrL4Async(string email)
{
    return (from m in this.Queryable()
            where m.Email == email
              && (m.RoleMemberships.Any(
                    r => r.RoleId == RoleConstants.ACCESSGRANTEDL1ID 
                      || r.RoleID == RoleConstants.ACCESSGRANTEDL3ID 
                      || r.RoleID == RoleConstants.ACCESSGRANTEDL4ID)
            select m).SingleOrDefaultAsync();
}

我猜以上内容会让事情变得更好。您可以使用调试器检查两种情况下生成的查询。

另外请注意，您不需要等待结果，只需返回任务，调用者就需要等待它。

Answer 2

我认为要加载的数据很多，而 EF 是一种非常慢的映射技术，它会创建有趣的查询（你可以看到这个here）。你有性能问题吗？也许你应该选择其他映射技术（堆栈例如Dapper)。当我们进行测试时，它的速度是 EF 的两倍。