性能问题案例02——sybase连接阻塞问题

现象:最近现场反馈一个问题,系统在审批的时候,经常卡死,整个系统完全用不了,浏览器访问处于loading的状态。



排查:

1.一般系统挂了首先想到内存问题,但是现象是loading,也就是说没有挂,线程正在执行,怀疑是线程被阻塞了,配置上jvisualvm监控了一下,出问题后内存没满确定不是内存问题,查看线程dump发现大部分都在执行sql查询,

初步发现是执行sql慢导致的。

 

2.我们用的是sybase数据库,执行了几个简单sql发现几分钟都没执行完,使用sp_sysmon "00:00:30"监控近30秒的情况发现cpu、内存、线程都没问题,几乎1%使用率都不到,怀疑是某个连接阻塞了表,导致其他连接全部阻塞导致的。

 

3.我们使用自己写的存储过程查看阻塞的连接,结果如下:

PS:使用sp_lock命令就可以查看哪个连接阻塞了数据库,但是显示的都是tableid等,还需要再次查询转换成具体表名等,自己写的存储过程只是此处转换了一下,后面会附上。

技术分享


发现其中1万多个锁,7000多个排它锁,Ex_row-blk是阻塞了其他连接的锁,发现有21个阻塞了其他连接的锁,对应的表是T_ZXLD_SYYH,执行select * from master..sysprocesses找到对应的连接,如上图的User有316/287/283等,结果如下:

技术分享


此时基本确定了原因,我们用的是c3p0连接池,应该是某个连接阻塞了表,其他所有连接查询时候都被阻塞了,导致连接池被占满,所有请求凡是涉及数据库查询的都被阻塞了,页面始终处于loading状态。

 

 4.那么接下来就是找到阻塞的地方,上图发现tran_name都是$chained_transaction,结合程序判断,也就是说都在执行某个责任链里面的事务时阻塞了,系统就2处使用了责任链,直接就可以判断到时审批的责任链导致的,那么接下来就是排查此处代码是否有问题了,审批流程如下:


(1)开启事务

(2)根据参数查询出要审批的主表数据,2个sql

(3)逐条调用审批组件(类似工作流的一个组件)审批,每条数据大约5个更新sql,2个查询sql

(4)更新主表状态,每条数据1个更新sql

(5)插入审计日志,每条数据1个插入sql

(6)生成提醒消息,查询全部主表数据,20个查询sql,5个更新删除sql(非常慢)

(7)更新增量记录表,记录该条数据修改时间和状态等,每条数据1个更新sql

(8)提交事务


怀疑并发时相互阻塞导致的,生成提醒消息的地方,会查询全部业务表,如果此时有其他连接在事务中审批,就会阻塞,其他连接在生成消息,相互阻塞,造成死锁。理论上数据库会自动处理死锁的,但是不知道什么原因,日志报的死锁数量不是特别多。


而且此处业务处理也太合理,正常用户每次批量审批大约100条数据,所以大约有900多个增删改sql,200多个查询sql,像生成消息等sql执行非常慢(因为涉及更新旧消息),这么多操作放到一个事务中非常慢,审批表大约3000w数据,业务主表500w数据,执行起来也不是很快。

 

解决办法:重构了此处代码,讲上面执行慢的(5)(6)(7)步骤新起一个线程,不放到事务中执行,即使失败也影响不大。这样sql减少了30%,速度快了不少,更新到现场,观察了一个星期,发现未出现不响应等情况,算是问题解决。

 

虽然问题解决,但是根本原因没有找到,为什么会相互阻塞,为什么死锁没自动检测,这个是后续需要跟踪的。

 

最后,问题虽然解决了,但是中间沟通花了不少时间,和运维人员要现场数据耽误了很多事,简单整理一下,后续出问题的时候搜集这几个数据:
1、sp__lock的结果;(使用文章最后提供的存储过程)
2、select * from master..sysprocesses;
3、如果数据库慢,加上sp_sysmon "00:00:30"的结果;
4、中间件线程dump文件;


附上sybase查看锁的存储过程

IF OBJECT_ID ('dbo.sp__lock') IS NOT NULL
	DROP PROCEDURE dbo.sp__lock
GO

create procedure sp__lock(
    @dbname char(30)=null,@spid int=null,
    @dont_format char(1) = null
)
as
begin

    declare @dbid smallint
    if @dbname is not null
        select @dbid=db_id(@dbname)

    if (charindex("sa_role", show_role()) > 0)
    begin
        if @dont_format is null
            select "Type"=substring(v.name,1,11),
                "User"=substring(suser_name(p.suid)+" ("+rtrim(convert(char(6),l.spid))+")",1,20),
                "Table"=substring(db_name(l.dbid)+".."+convert(char(20),object_name(l.id,l.dbid)),1,26),
                "Page"=convert(char(8),l.page),
                "Cmd"=substring(p.cmd,1,11)
            from master..syslocks l,
                 master..sysprocesses p,
                 master..spt_values v
            where p.spid=l.spid and
                l.type = v.number and
                v.type = "L" and
                p.dbid=isnull(@dbid,p.dbid) and
                p.spid=isnull(@spid,p.spid) and
                l.dbid=isnull(@dbid,l.dbid) and
                l.spid=isnull(@spid,l.spid)
            order by l.dbid, l.id, v.name
        else
            select "Type"=v.name,
                "User"=suser_name(p.suid)+" ("+rtrim(convert(char(6),l.spid))+")",
                "Table"=db_name(l.dbid)+".."+object_name(l.id,l.dbid),
                "Page"=l.page,
                "Cmd"=p.cmd
            from master..syslocks l,
                  master..sysprocesses p,
                 master..spt_values v
            where p.spid=l.spid and
                l.type = v.number and
                v.type = "L" and
                p.dbid=isnull(@dbid,p.dbid) and
                p.spid=isnull(@spid,p.spid) and
                l.dbid=isnull(@dbid,l.dbid) and
                l.spid=isnull(@spid,l.spid)
            order by l.dbid, l.id, v.name
        return
    end

    select "Type"=v.name,
        "Usernm"=convert(varchar(60),suser_name(p.suid)+" ("+rtrim(convert(char(6),l.spid))+")"),
        "TableNm"=convert(varchar(60),db_name(l.dbid)+".."),
        "Page"=l.page,
        "Cmd"=p.cmd,
        l.id,
        l.dbid
    into #locks
    from master..syslocks l,
         master..sysprocesses p,
         master..spt_values v
    where p.spid=l.spid    and
        l.type = v.number    and
        v.type = "L" and
        l.dbid=isnull(@dbid,l.dbid) and
        l.spid=isnull(@spid,l.spid) and
        p.dbid=isnull(@dbid,p.dbid) and
        p.spid=isnull(@spid,p.spid)

    update #locks
    set    TableNm=TableNm+object_name(id,dbid)
    where dbid=db_id() or dbid=1 or dbid=2

    update #locks
    set TableNm=TableNm+convert(varchar,id)
    where dbid<>db_id() and dbid>2

    delete #locks
    where TableNm like "tempdb..#locks%"

    if @dont_format is null
        select substring(Type, 1,11),
            "User"=substring(Usernm, 1,14),
            "Table"=convert(char(26),TableNm),
            "Page"=convert(char(8),Page),
            "Cmd"=substring(Cmd,1,11)
        from #locks
        order by dbid, id, Type
    else
        select Type, "User"=Usernm, "Table"=TableNm, Page, Cmd
        from #locks
        order by dbid, id, Type

end
GO



郑重声明:本站内容如果来自互联网及其他传播媒体,其版权均属原媒体及文章作者所有。转载目的在于传递更多信息及用于网络分享,并不代表本站赞同其观点和对其真实性负责,也不构成任何其他建议。