Marten项目异步投影快照双重处理事件的Bug分析

背景介绍

Marten是一个基于PostgreSQL的.NET事件存储和文档数据库库。在最新发布的7.x版本中，引入了一个关于异步快照投影(Async Snapshot Projection)的重要Bug，该Bug会导致事件被双重处理，进而影响投影状态的正确性。

问题现象

当使用异步快照投影时，Marten会错误地对同一个事件进行两次处理。具体表现为：

1. 对于新创建的流(stream)，第一个事件会被投影处理两次
2. 第一次处理会正确设置初始状态
3. 第二次处理会再次尝试更新投影，但由于版本号冲突，实际更新被跳过
4. 结果导致后续事件处理时，投影状态不是最新的

技术细节分析

这个Bug的核心在于Marten 7.x的异步守护进程(Daemon)在处理新流的事件时存在逻辑错误。当守护进程获取事件切片(event slice)进行处理时，对于新创建的流，它会错误地将同一个事件放入处理队列两次。

在底层实现上，Marten使用PostgreSQL的UPSERT函数来更新投影状态。这个函数包含版本控制逻辑：

if revision = 1 then
  SELECT mt_version FROM mt_doc_projection into current_version WHERE id = docId;
  if current_version is not null then
    revision = current_version + 1;
  end if;
end if;

当同一个事件被处理两次时，第一次处理会创建投影记录并设置版本号为1。第二次处理时，由于记录已存在，版本号会被递增到2。这导致后续真正应该处理版本2的事件时，更新操作会被跳过，因为版本号已经"超前"了。

影响范围

这个Bug影响所有使用以下配置的Marten 7.x用户：

options.Projections.Snapshot<TProjection>(SnapshotLifecycle.Async);

其中TProjection是任何自定义的投影类。受影响的主要是新创建的流，已有流的处理不受影响。

解决方案

Marten团队已经修复了这个Bug。修复的核心是确保异步守护进程正确处理事件切片，避免对同一事件进行重复处理。

对于无法立即升级的用户，可以考虑以下临时解决方案：

1. 修改UPSERT函数逻辑，将版本检查从">"改为">="（不推荐，可能有副作用）
2. 禁用投影的版本强制检查（需要小心处理，可能引入其他问题）

最佳实践

为了避免类似问题，建议：

1. 为关键业务逻辑编写集成测试，验证投影状态是否符合预期
2. 在升级Marten版本后，全面测试投影功能
3. 监控投影处理日志，关注异常情况

总结

这个Bug展示了在复杂事件处理系统中，即使是微小的逻辑错误也可能导致严重的数据一致性问题。Marten团队快速响应并修复了这个问题，体现了开源社区的高效协作。对于使用Marten的开发者来说，理解投影机制的工作原理和版本控制策略，有助于更快地识别和解决类似问题。