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

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

dba-kit · 2024 年7 月 6 日 14:09

额，确实是，只放到建议反馈里了。。

TiDB_M · 2024 年7 月 7 日 07:18

一直都存在的bug呀无伤大雅

jhm633 · 2024 年7 月 8 日 10:46

是已知问题，可以用这个 issue 跟踪进度 Issues · pingcap/tidb · GitHub

dba-kit · 2024 年7 月 10 日 02:47

现在更夸张了，245结尾那个 tidb-server 节点，default 的总连接数竟然已经是-1724了。。

YangKeao · 2024 年7 月 10 日 09:00

如果连接在完全建立前退出，可能会导致 Connection Count 减一，使得它比实际的值小。很容易的复现方式是：使用 mysql cli 连接一个不存在的用户（或使用错误的密码连接用户），TiDB 的 Connection Count 就会减一比实际值小。

这个问题有个 PR 在修了！https://github.com/pingcap/tidb/pull/54546

MrSylar · 2024 年7 月 10 日 09:49

学习了

T02iDBer_7S8XqKfl · 2024 年7 月 11 日 08:36

期待官方的解决方案

dba-kit · 2024 年9 月 9 日 08:37

此话题已在最后回复的 60 天后被自动关闭。不再允许新回复。