On 14.12.12 14:20, Alex Parvulescu wrote:
> ouch, maybe my index pruning mechanism isn't too smart after all :(

I don't think so. It's more likely something with the memory node 
builder, which has just been uncovered by your changes. See 
https://issues.apache.org/jira/browse/OAK-520

Michael

>
> I'll take anther look.
>
> akex
>
> On Fri, Dec 14, 2012 at 3:13 PM, Michael Dürig <mduerig@apache.org> wrote:
>
>>
>>
>> On 14.12.12 14:07, Angela Schreiber wrote:
>>
>>> the failing commit should have only fixed code indention.
>>> i don't understand how that can impact on AuthorizablePropertyTest.
>>>
>>
>> Seems to be a transient issue. Caused by the new p2 index. Here's the
>> stack trace I've seen locally.
>>
>> Michael
>>
>> java.lang.**IllegalStateException
>>          at com.google.common.base.**Preconditions.checkState(**
>> Preconditions.java:133)
>>          at org.apache.jackrabbit.oak.**plugins.memory.**
>> MemoryNodeBuilder.read(**MemoryNodeBuilder.java:205)
>>          at org.apache.jackrabbit.oak.**plugins.memory.**MemoryNodeBuilder.
>> **getChildNodeNames(**MemoryNodeBuilder.java:379)
>>          at org.apache.jackrabbit.oak.**plugins.index.p2.strategy.**
>> ContentMirrorStoreStrategy.**remove(**ContentMirrorStoreStrategy.**
>> java:66)
>>          at org.apache.jackrabbit.oak.**plugins.index.p2.**
>> Property2IndexUpdate.apply(**Property2IndexUpdate.java:143)
>>          at org.apache.jackrabbit.oak.**plugins.index.p2.**
>> Property2IndexDiff.apply(**Property2IndexDiff.java:232)
>>          at org.apache.jackrabbit.oak.**plugins.index.**
>> IndexHookManager.apply(**IndexHookManager.java:71)
>>          at org.apache.jackrabbit.oak.**plugins.index.**IndexHookManager.**
>> processCommit(**IndexHookManager.java:61)
>>          at org.apache.jackrabbit.oak.spi.**commit.CompositeHook.**
>> processCommit(CompositeHook.**java:59)
>>          at org.apache.jackrabbit.oak.**kernel.KernelNodeStoreBranch.**
>> merge(KernelNodeStoreBranch.**java:127)
>>          at org.apache.jackrabbit.oak.**core.RootImpl$2.run(RootImpl.**
>> java:240)
>>          at org.apache.jackrabbit.oak.**core.RootImpl$2.run(RootImpl.**
>> java:236)
>>          at java.security.**AccessController.doPrivileged(**Native Method)
>>          at javax.security.auth.Subject.**doAs(Subject.java:337)
>>          at org.apache.jackrabbit.oak.**core.RootImpl.commit(RootImpl.**
>> java:235)
>>          at org.apache.jackrabbit.oak.jcr.**SessionDelegate.save(**
>> SessionDelegate.java:255)
>>          at org.apache.jackrabbit.oak.jcr.**SessionImpl.save(SessionImpl.**
>> java:283)
>>          at org.apache.jackrabbit.oak.jcr.**security.user.**
>> AbstractUserTest.tearDown(**AbstractUserTest.java:72)
>>          at org.apache.jackrabbit.test.**AbstractJCRTest.run(**
>> AbstractJCRTest.java:456)
>>          at org.junit.internal.runners.**JUnit38ClassRunner.run(**
>> JUnit38ClassRunner.java:83)
>>          at org.junit.runner.JUnitCore.**run(JUnitCore.java:157)
>>          at com.intellij.junit4.**JUnit4IdeaTestRunner.**
>> startRunnerWithArgs(**JUnit4IdeaTestRunner.java:76)
>>          at com.intellij.rt.execution.**junit.JUnitStarter.**
>> prepareStreamsAndStart(**JUnitStarter.java:195)
>>          at com.intellij.rt.execution.**junit.JUnitStarter.main(**
>> JUnitStarter.java:63)
>>          at sun.reflect.**NativeMethodAccessorImpl.**invoke0(Native Method)
>>          at sun.reflect.**NativeMethodAccessorImpl.**invoke(**
>> NativeMethodAccessorImpl.java:**39)
>>          at com.intellij.rt.execution.**application.AppMain.main(**
>> AppMain.java:120)
>>
>>
>>
>