0

我正在寻找有关我们在应用程序中使用的存储过程的一些评论。它被称为很多，我认为还有改进的余地。我也在寻找向团队和 Opp 添加索引是否会对 SP 有所帮助。

我们在 Azure DB 上运行它。

该表的架构如下：

CREATE TABLE [dbo].[TeamHistoryMatchUps] (
    [Id]              UNIQUEIDENTIFIER DEFAULT (newid()) NOT NULL,
    [Team]            NVARCHAR (100)   NOT NULL,
    [Opp]             NVARCHAR (100)   NOT NULL,
    [Result]          INT              NOT NULL,
    [MatchResultTime] DATETIME2 (7)    DEFAULT (getdate()) NOT NULL,
    PRIMARY KEY CLUSTERED ([Id] ASC)
);

这是 SP：

CREATE PROCEDURE [dbo].[up_GetTeamPercentagev2]
@Team NVARCHAR(100),
@Opp NVARCHAR(100) 
AS
begin
set nocount ON
declare

@TotalResult INT,
@TeamResult INT


--Total Matchups
Set  @TotalResult = (SELECT count(*)  FROM TeamHistoryMatchUps
WHERE (Team = @Team OR Opp = @Team) AND (Team = @Opp OR Opp = @Opp)
AND Result = 1)

Set  @TeamResult = (SELECT COUNT(*) FROM TeamHistoryMatchUps
WHERE Team = @Team and Opp = @Opp
AND Result = 1)

SELECT (@TeamResult * 100 / @TotalResult) AS Percentage

exit_proc:
end

我应该提到，我担心插入，因为就在调用这个 sp 之前，将插入插入到表中，然后调用以获得随着时间的推移在这场比赛中的获胜百分比。

在使用了几次显示执行计划后，我确实添加了两个非聚集索引。

GO
CREATE NONCLUSTERED INDEX [[IX_MatchUps]
    ON [dbo].[TeamHistoryMatchUps]([Result] ASC)
    INCLUDE([Team], [Opp]);
GO
CREATE NONCLUSTERED INDEX [IX_MatchupsTeamOpp]
    ON [dbo].[TeamHistoryMatchUps]([Team] ASC, [Opp] ASC)
    INCLUDE([Result], [MatchResultTime], [MatchUpId]);

该表将进入百万行。目前在120k左右。

我在每个团队的 TeamHistoryMatchUps 中添加了 2 条记录，结果为 0 或 1。我试图保持它非常简单，以便可以进行上述查询。

CREATE PROCEDURE [dbo].[up_GetTeamPercentage]
    @Team NVARCHAR(100),
    @Opp NVARCHAR(100)
AS
SELECT 
    SUM(SIGN(result)) * 100 / COUNT(*) 
    AS Percentage
    FROM TeamHistoryMatchUps
    WHERE Team = @Team AND Opp = @Opp

但是认为更少的写入和更复杂的读取（在 SP 中）会是更好的方法。

4

4 回答 4

1

如果您不担心插入速度很慢，我会说继续添加索引以获得更好的选择性能。

索引也应该过滤结果为 1 的结果。

CREATE NONCLUSTERED INDEX [IX_TeamHistoryMatchUps_team_opp] 
ON [dbo].[TeamHistoryMatchUps] ([Team],[Opp])
WHERE result=1
于 2017-04-14T12:54:56.253 回答
0

我认为这应该会减少访问（同时，我想看看是否可以只使用一个 SELECT）。建议的索引应该会有所帮助。

--Total Matchups
SELECT @TeamResult = COUNT(*) FROM TeamHistoryMatchUps
WHERE  Result = 1
    AND Team = @Team and Opp = @Opp

SELECT @TotalResult = count(*)  
FROM TeamHistoryMatchUps
WHERE Opp = @Team AND Team = @Opp 
AND Result = 1

SET @TotalResult= @TotalResult+@TeamResult
于 2017-04-14T12:56:17.603 回答
0

答案是它取决于 TeamHistoryMatchUps 表中的记录数，以及每个列中有多少不同的值。如果表中没有大量记录，查询优化器可能仍会创建一个涉及索引扫描的执行计划（它读取索引中的每个叶子记录以查找匹配项）。这并不比全表扫描快多少。

如果有很多记录并且您在团队中搜索的值和 opp 索引将返回大约 15% 或更少的行，则查询优化器可能会选择通过索引查找来使用索引。在这种情况下，性能会有所提高。

于 2017-04-14T14:22:24.787 回答
0

将 sproc 参数重新声明为本地参数。它处理参数嗅探，当 WITH RECOMPILE 没有帮助时，这也提高了性能（来源：TBD）

CREATE PROCEDURE [dbo].[up_GetTeamPercentagev2] @Team NVARCHAR(100), @Opp NVARCHAR(100) AS begin set nocount ON declare

@localTeam NVARCHAR(100), @localOpp NVARCHAR(100), @TotalResult INT, @TeamResult INT

Set @Team = @localTeam SET @Opp = @localOpp

--Total Matchups Set @TotalResult = (SELECT count(*) FROM TeamHistoryMatchUps WHERE (Team = @localTeam OR Opp = @localTeam) AND (Team = @localOpp OR Opp = @localOpp) AND Result = 1)

Set @TeamResult = (SELECT COUNT(*) FROM TeamHistoryMatchUps WHERE Team = @localTeam and Opp = @localOpp AND Result = 1)

SELECT (@TeamResult * 100 / @TotalResult) AS Percentage

exit_proc: end

***从未在 Azure DB 上尝试过

于 2017-04-14T18:50:08.463 回答