Cacti项目中MySQL表锁问题的分析与解决

问题背景

在Cacti 1.3.0开发版本中，开发团队发现了一个与数据库表锁相关的严重问题。该问题最初是在修复另一个bug（编号5812）后出现的，表现为系统日志中频繁出现"CMDPHP ERROR: A DB Row Failed!, Error: Table 'po' was not locked with LOCK TABLES"的错误信息。

错误表现

系统日志中主要出现两类错误：

1. 表未锁定错误：系统报告"Table 'po' was not locked with LOCK TABLES"，表明在尝试访问poller_output_boost表（别名po）时，该表未被正确锁定。

2. SQL语法错误：系统报告"Error: You have an error in your SQL syntax...near 'LOCK TABLE data_local AS dl READ LOCAL'"，表明LOCK TABLE语句的语法存在问题。

这些错误主要出现在WeatherMap插件尝试读取RRD数据时，通过boost_fetch_cache_check函数调用链触发。

技术分析

根本原因

问题的核心在于MySQL/MariaDB的表锁定机制。在最初的修复尝试中，开发团队使用了以下锁定方式：

if ($locks) {
    db_execute("LOCK TABLE poller_output_boost AS po READ LOCAL");
    db_execute("LOCK TABLE data_local AS dl READ LOCAL");
}

这种实现存在两个问题：

1. 语法不兼容：MySQL 8.0及MariaDB要求使用"LOCK TABLES"（复数形式）语法，而不是"LOCK TABLE"（单数形式）。

2. 锁定范围不足：在多表查询场景下，需要同时锁定所有相关表，而不是分别锁定。

解决方案演进

开发团队经过多次迭代，最终确定了以下解决方案：

1. 统一使用LOCK TABLES语法：将单数形式的LOCK TABLE改为复数形式的LOCK TABLES。

2. 合并锁定语句：将两个独立的锁定语句合并为一个复合语句，确保原子性：

if ($locks) {
    db_execute("LOCK TABLES poller_output_boost AS po READ LOCAL, data_local AS dl READ LOCAL");
}

3. 引入临时表机制：对于高并发场景，开发团队还引入了临时表机制来缓解锁定冲突。

4. 错误处理增强：增加了对临时表创建失败的容错处理。

影响范围

该问题主要影响：

1. 使用MySQL 8.0或MariaDB的环境
2. 启用了Boost插件和WeatherMap插件的Cacti系统
3. 高频率数据采集场景

解决方案验证

经过多次测试验证，最终方案能够：

1. 正确锁定相关表
2. 避免SQL语法错误
3. 保持系统稳定性
4. 兼容不同版本的MySQL/MariaDB

最佳实践建议

对于Cacti管理员：

1. 确保使用最新版本的修复补丁
2. 定期检查系统日志中的数据库错误
3. 在高负载环境中考虑调整数据采集频率
4. 对于大型部署，建议进行充分的测试后再应用生产环境

该问题的解决体现了Cacti开发团队对系统稳定性的高度重视，也展示了开源社区协作解决问题的效率。通过这次修复，Cacti的数据处理可靠性和兼容性得到了进一步提升。