【发布时间】:2018-09-29 22:20:11
【问题描述】:
在 RavenDB 4 (v4.0.3-patch-40031) 中,我有两种文档类型:Apple 和 Orange。两者都有相似但也不同的属性。我在运行时的代码中遇到了一个错误,有时会提供 Apple 的 ID,但会返回 Orange。可怕的!
深入研究它,这有点道理。但我正在努力寻找合适的解决方案。
来了。在 RavenDB 中,我将单个 Apple 存储为文档:
id: "078ff39b-da50-4405-9615-86b0d185ba17"
{
"Name": "Elstar",
"@metadata": {
"@collection": "Apples",
"Raven-Clr-Type": "FruitTest.Apple, FruitTest"
}
}
为了这个例子,假设我没有Orange 文档存储在数据库中。我希望这个测试能够成功:
// arrange - use the ID of an apple, which does not exist in Orange collection
var id_of_apple = "078ff39b-da50-4405-9615-86b0d185ba17";
// act - load an Orange
var target = await _session.LoadAsync<Orange>("078ff39b-da50-4405-9615-86b0d185ba17");
// assert - should be null, because there is no Orange with that Id
target.Should().BeNull(because: "provided ID is not of an Orange but of an Apple");
...但它失败了。发生的情况是文档 ID 存在,因此 RavenDB 会加载文档。不在乎是什么类型。它会尝试自动映射属性。我预期或错误地假设 Load 类型说明符会将查找限制为该特定文档集合。相反,它在整个数据库中抓取 + 映射它,而不是将其限制为 type <T>。所以行为不同于.Query<T>,后者对收集有约束。
需要注意的是,我使用 guids 作为身份策略,将 Id 设置为
string.Empty(符合 the docs)。我假设默认的 ID 策略,比如entityname/1001,不会有这个问题。
docs on Loading Entities 并没有真正提及这是否是故意的。它只是说:“从数据库下载文档并将它们转换为实体。”。
但是,出于某些原因,我确实希望将 Load 操作限制为单个集合。或者,更好的说法是,尽可能高效地按 ID 从特定集合中加载文档。如果不存在,则返回null。
AFAIK,有两种方法可以实现:
- 使用更昂贵的
.Query<T>.Where(x => x.Id == id),而不是.Load<T>(id) - 首先执行
.Load<T>(id),然后检查(~不知何故,见底部)它是否是集合 T 的一部分
我的问题可以总结为两个问题:
- 还有比上述两个选项更高效或更稳定的方法吗?
- 如果没有,从两个选项中选择哪一个 - 从性能和稳定性方面推荐?
特别是对于第二个问题,很难正确地正确衡量这一点。至于稳定性,例如没有副作用,我想对 RavenDB 内部结构有更深入了解或经验的人可能会对此有所了解。
注意该问题假定所解释的行为是故意的,而不是 RavenDB 错误。
~不知何故会是:
public async Task<T> Get(string id)
{
var instance = await _session.LoadAsync<T>(id);
if (instance == null) return null;
// the "somehow" check for collection
var expectedTypeName = string.Concat(typeof(T).Name, "s");
var actualTypeName = _session.Advanced.GetMetadataFor(instance)[Constants.Documents.Metadata.Collection].ToString();
if (actualTypeName != expectedTypeName)
{
// Edge case: Apple != Orange
return null;
}
return instance;
}
如何重现
2018 年 4 月 19 日更新 - 在有用的 cmets 之后添加了这个可重现的样本(谢谢)。
模型
public interface IFruit
{
string Id { get; set; }
string Name { get; set; }
}
public class Apple : IFruit
{
public string Id { get; set; }
public string Name { get; set; }
}
public class Orange : IFruit
{
public string Id { get; set; }
public string Name { get; set; }
}
测试
例如。在同一个会话中抛出 InvalidCastException(有效),但在第二次它没有。
public class UnitTest1
{
[Fact]
public async Task SameSession_Works_And_Throws_InvalidCastException()
{
var store = new DocumentStore()
{
Urls = new[] {"http://192.168.99.100:32772"},
Database = "fruit"
}.Initialize();
using (var session = store.OpenAsyncSession())
{
var apple = new Apple
{
Id = Guid.NewGuid().ToString(),
Name = "Elstar"
};
await session.StoreAsync(apple);
await session.SaveChangesAsync();
await Assert.ThrowsAsync<InvalidCastException>(() => session.LoadAsync<Orange>(apple.Id));
}
}
[Fact]
public async Task Different_Session_Fails()
{
var store = new DocumentStore()
{
Urls = new[] {"http://192.168.99.100:32772"},
Database = "fruit"
}.Initialize();
using (var session = store.OpenAsyncSession())
{
var appleId = "ca5d9fd0-475b-41de-a1ab-57bb1e3ce018";
// this *should* break, because... it's an apple
// ... but it doesn't - it returns an ORANGE
var orange = await session.LoadAsync<Orange>(appleId);
await Assert.ThrowsAsync<InvalidCastException>(() => session.LoadAsync<Orange>(appleId));
}
}
}
【问题讨论】:
-
对我来说它抛出了一个 Cast 异常,你确定它没有失败吗?
-
@Embri 是的,100% 肯定。它不会在抛出错误的意义上失败——它实际上返回了错误的文档类型(映射了属性)。我在 ~Somehow 上添加了一个示例 - 这可能会更好地解释它。同样,请注意 ID 生成策略与默认策略不同。
-
还需要注意的是,Apples 和 Oranges 具有相似的属性。这也许就是我没有收到 Cast 异常的原因? @Embri
-
事实上我不明白为什么你没有收到那个异常..Apples 和 Oranges 是同一个基类的子类?
-
不,它们派生自接口(以 Id 作为属性),但没有基类。刚刚测试过,但 InvalidCastException 仅在您执行 Store + Get 在同一会话中时发生。如果您在新会话中使用现有 ID 执行 LoadAsync
,我的问题应该会重现。