sorry for late reply , i could not get notification of the update ..

I took the following compass changes for the fix and did a build

• /trunk/src/main/src/org/compass/core/config/binding/PlainMappingBinding.java (version 3922)
• /trunk/src/main/test/org/compass/core/test/json/object/mapping.cpm.xml (version 3922)

i could see that you have made a code change in the file PlainMappingBinding.java and have included the above reported structure in the file mapping.cpm.xml. as rootobject "d" .....
The tests are all passing but i did not see the file "mapping.cpm.xml" being used anywhere ( for initialisation for indexing)...
Instead, the file SimpleJsonObjectBuilderTests.java uses the following code to initialise the mapping

conf.addMapping(
json("c")
.add(id("id"))
.add(property("value"))
.add(object("obj").add(property("objValue1").namingType(Naming.FULL).valueConverter("int")).add(array("arr").element(property().namingType(Naming.FULL).valueConverter("int"))))

Now ,
Here if we add the mapping for object "d" as
conf.addMapping(
json("d")
.add(id("id"))
.add(object("employee").add(property("guid")).add(property("version")).add(property("creationdate")).add(object("address").add(property("street"))))
.add(content("content"))

the test will pass .....
but if i use the mapping file instead and add the mapping using :
config = config.configure().addFile(<path to mapping.cpm.xml>);
then compass goes to stack overflow error( because of the rootobject "d" which has an object inside object )

Conclusion: if we make the mapping using programmatic builder API ( eg , conf.addMapping(json(d).add ....... ) , then a structure which has an object inside object works well ... but if we make the same mapping using fully explicit mappings ( eg <json-rootobject alias="d" ... ) then compass goes into stack overflow error.

The error still persists..

Please let me know if you need any more information.

Regards

Just for kicks, I added the same programmatic to SimpleJsonObjectBuilderTests and it passes as well. I also added an actual test to verify the indexing work.

Can you simply double check with trunk?

the problem still seems to persist.
Whenever i use config = config.configure().addFile(<path to mapping.cpm.xml>); , compass goes to stack overflow.

btw, in the change log i could find the following files that are changed, but all the below 3 files looks like to be part of test suite and not part of the fix ... am i missing something ???

• /trunk/src/main/test/org/compass/core/test/json/object/SimpleJsonObjectTests.java (version 3923)
• /trunk/src/main/test/org/compass/core/test/json/object/SimpleJsonObjectBuilderTests.java (version 3923)
• /trunk/src/main/test/org/compass/core/test/json/object/mapping.cpm.xml (version 3923)

The above 3 files are the only changes that i took from yestarday's build .. It is very likely that i might missing the files which have the actual fix..
Please let me know the actual files to pick for the fix ..

Thanks and Regards

This is very strange. The changes I did are indeed just changes to the testing to test the mapping you gave more properly. When I run the tests, they pass, so I don't really understand what does not pass for you.

Shay

I really appologise for the inconvenience caused ...iam not getting the stack overflow error now ... I was using a wrong value for classpath( which was taking compass2.2.0 instead of your trunk build )

Thanks a lot for your time on the issue ..
Should i close the bug 898 ?

Thanks a lot again....

Regards