Static indexes and a common base class with Id property fails for build 888

47 views
Skip to first unread message

Asger Hallas

unread,
Apr 16, 2012, 11:48:04 AM4/16/12
to rav...@googlegroups.com
Hi

I have a problem where a field in a static index is wrongly pointing to __document_id when it is created.

I have a failing test for it here: http://pastebin.com/ujY4uW9q

It fails only if:

1. It is a static index
2. I have a common base class with the Id property for the root and its child

//Asger

Oren Eini (Ayende Rahien)

unread,
Apr 17, 2012, 6:19:17 AM4/17/12
to rav...@googlegroups.com
This is a bug in the way we are generating indexes.
Fixed now

Asger Hallas

unread,
Apr 18, 2012, 4:48:25 AM4/18/12
to rav...@googlegroups.com
Thanks!

I have now tried it out on unstable from today, but the test still fails, albeit with another error:

System.InvalidCastException: Unable to cast object of type 'ICSharpCode.NRefactory.Ast.IdentifierExpression' to type 'ICSharpCode.NRefactory.Ast.InvocationExpression'.

Is this known or am I doing it wrong?

Oren Eini (Ayende Rahien)

unread,
Apr 18, 2012, 5:02:34 AM4/18/12
to rav...@googlegroups.com
Huh? 
Can you show a full failing test? 

Asger Hallas

unread,
Apr 18, 2012, 5:32:55 AM4/18/12
to rav...@googlegroups.com
Mmm... sorry that exception was my mistake. I ran IndexCreation.CreateIndexes over the entire test assembly :)

But the original test still fails. I have made a pull request with test in it here https://github.com/ravendb/ravendb/pull/473

Oren Eini (Ayende Rahien)

unread,
Apr 18, 2012, 6:51:40 AM4/18/12
to rav...@googlegroups.com
You are correct, that was an issue in 910.
I pushed the commit fixing this to 1.2 by accident
I pulled it back in and it will be in the next build

Asger Hallas

unread,
Apr 18, 2012, 7:58:29 AM4/18/12
to rav...@googlegroups.com
Thanks a lot! :)
Reply all
Reply to author
Forward
0 new messages