Prometheus Java客户端中System.properties并发访问问题解析与解决方案

问题背景

在Java应用监控领域，Prometheus Java客户端库被广泛使用。该库在初始化配置时会读取系统属性(System.properties)，但在多线程环境下，这一操作存在潜在风险。系统属性作为JVM级别的全局共享状态，其底层实现是Hashtable类型，虽然单个操作是线程安全的，但在遍历操作时仍可能引发并发修改异常(ConcurrentModificationException)。

问题本质分析

异常发生的核心原因在于：

1. System.properties使用Hashtable存储，虽然put/get操作有同步保证，但迭代器遍历时没有完全解决并发修改问题
2. 当PrometheusPropertiesLoader加载配置时，直接调用System.getProperties().entrySet()进行遍历复制
3. 若此时其他线程同时修改系统属性，就会破坏迭代器的一致性状态

技术细节剖析

Hashtable的并发控制机制存在局限性：

• 方法级别的synchronized保证了单个操作的原子性
• 但复合操作（如迭代+修改）仍需要外部同步
• 这与ConcurrentHashMap的设计哲学不同，后者提供了更细粒度的并发控制

解决方案演进

原始方案的问题

new HashMap(System.getProperties())  // 直接基于Properties构造新Map

这种写法隐式调用了Map.putAll()，底层会遍历原始Properties的entrySet，在多线程环境下不安全。

改进方案一：同步代码块

Map<String, String> properties;
synchronized (System.getProperties()) {
    properties = new HashMap(System.getProperties());
}

通过外部同步确保遍历时的线程安全，但会带来性能损耗。

改进方案二：键集合复制

Properties systemProperties = System.getProperties();
Set<String> keys;
synchronized (systemProperties) {
    keys = new HashSet(systemProperties.stringPropertyNames());
}
Map<String, String> properties = new HashMap<>();
for (String key : keys) {
    properties.put(key, systemProperties.getProperty(key));
}

先复制键集合再逐个获取值，减少同步块范围。

最优方案：使用stringPropertyNames

Map<String, String> properties = new HashMap<>();
for (String key : System.getProperties().stringPropertyNames()) {
    properties.put(key, System.getProperty(key));
}

利用stringPropertyNames()方法返回键集合的快照，避免直接遍历entrySet。

最佳实践建议

1. 对于配置加载等初始化操作，建议采用键集合复制方案
2. 在高并发场景下，考虑缓存系统属性副本避免频繁读取
3. 使用防御性编程，对可能并发的属性访问添加重试机制
4. 在Java 8+环境中，可考虑使用ConcurrentHashMap代替系统属性副本

扩展思考

这个问题反映了Java系统属性设计的时代局限性。在现代Java开发中：

• 更推荐使用专门的配置管理框架
• 对于需要频繁读取的配置，应采用immutable设计模式
• 考虑使用环境变量替代部分系统属性配置
• 在微服务架构中，建议集中式配置中心替代本地配置

通过这个案例，开发者可以深入理解Java并发编程中的可见性、原子性和一致性等核心概念，以及如何在日常开发中避免类似的并发陷阱。