Marten项目中的Linq查询运算符.In在QueryAllRawEvents中的回归问题分析

问题背景

Marten是一个.NET平台上的PostgreSQL文档数据库和事件存储库。在最新版本(v7.5)中，开发团队发现了一个关于Linq查询运算符.In在.QueryAllRawEvents()方法中的行为变化，这导致了SQL生成的不正确。

问题现象

在v6.4版本中，当开发者使用以下查询时：

session.Events.QueryAllRawEvents()
    .Where(e => e.Id.In(eventsIds))
    .Where(e => e.AnyTenant());

生成的SQL会正确地查询mt_events表中的id列：

select data, type, mt_dotnet_type, seq_id, id, stream_id, version, timestamp, tenant_id, is_archived 
from public.mt_events as d 
where ((d.id = ANY(:p0) and 1=1) and d.is_archived = FALSE

但在升级到v7.5后，同样的查询会生成不同的SQL，错误地从JSON数据字段(data)中提取ID值：

select data, type, mt_dotnet_type, seq_id, id, stream_id, version, timestamp, tenant_id, is_archived 
from public.mt_events as d 
where (1=1 and CAST(d.data ->> 'Id' as uuid) = ANY(:p0) and d.is_archived = FALSE

技术分析

.QueryAllRawEvents()方法的设计初衷是查询事件存储中的原始事件记录，包括事件元数据(如ID、流ID、版本等)和事件数据本身。根据Marten的设计原则：

1. 事件元数据(如ID)应该直接映射到数据库表的列
2. 事件内容则存储在JSON格式的data列中

在v6.4版本中，.In运算符正确地处理了IEvent接口的属性，直接查询数据库列。但在v7.5版本中，查询逻辑错误地转向了从JSON数据中提取值，这显然不符合.QueryAllRawEvents()方法的语义。

影响范围

这个问题会影响所有使用以下组合的应用程序：

1. 使用.QueryAllRawEvents()方法
2. 在查询中使用.In运算符过滤事件ID
3. 从v6.4升级到v7.5版本

解决方案

Marten开发团队已经确认这是一个bug，并在最新提交中修复了这个问题。修复方案是确保.In运算符在.QueryAllRawEvents()上下文中正确地处理IEvent接口的属性，而不是从JSON数据中提取值。

最佳实践

对于使用Marten事件存储的开发者，建议：

1. 在升级版本时，仔细测试所有事件查询
2. 对于关键查询，考虑编写单元测试验证生成的SQL是否符合预期
3. 关注Marten的发布说明，了解行为变更

总结

这个案例展示了ORM框架中Linq提供程序实现的重要性，以及版本升级时可能遇到的微妙行为变化。Marten团队快速响应并修复了这个回归问题，体现了开源项目的活跃维护状态。开发者在使用高级查询功能时，应当理解底层生成的SQL，以确保查询行为的正确性。