tidb_server_connections存在小于0的值

dba-kit · 2024 年6 月 27 日 07:18

升级7.5.2后，发现有些场景下，监控里的 tidb-server 总连接数竟然是个负数。

WinterLiu · 2024 年7 月 1 日 03:28

奇怪了，难道是BUG或者是理解不对吗

有猫万事足 · 2024 年7 月 1 日 03:41

github.com/pingcap/tidb

Connection count metric can be less than the real value

opened 07:14AM - 19 Mar 24 UTC

closed 09:47AM - 22 Mar 24 UTC

YangKeao

type/bug sig/sql-infra severity/moderate affects-7.5 found/community

This issue has similar reason with #42761, because the `conn.Close()` will be ca…lled multiple times :facepalm:. ```go func (s *Server) onConn(conn *clientConn) { ... defer func() { terror.Log(conn.Close()) logutil.Logger(ctx).Debug("connection closed") }() ... } ``` And ```go func (cc *clientConn) Run(ctx context.Context) { defer func() { ... if cc.getStatus() != connStatusShutdown { err := cc.Close() terror.Log(err) } close(cc.quit) }() ``` We have two ways to solve this problem: 1. Don't include any logic in `conn.Close()`. Only have logic in the `sync.Once` inside the `closeConn`. 2. Enhance the code, **don't close the connection for multiple times!** (it's more intuitive for the developers).

7.5.2以前确实有这个bug。但是修复时间看应该7.5.2就没有了。
也许这些错误的数据是在升级时间以前？
不知道是否如此。如果还是错的，在issue里面反馈一下看看。

dba-kit · 2024 年7 月 4 日 03:21

额，这个 BUG也是我提的，之前更夸张基本都为0。另外今天来看，值已经变成-65了，估计会一直累积。。

TIDB-Learner · 2024 年7 月 4 日 03:34

连接是0 也不能负数。

Billmay表妹 · 2024 年7 月 4 日 07:16

你忘记放到产品缺陷分类了！