[ 
https://issues.apache.org/jira/browse/OAK-4954?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Andrei Dulceanu updated OAK-4954:
---------------------------------
    Attachment: OAK-4954-02.patch

[~mduerig] I re-worked the patch based on my previous find that keeping the 
test node on a per thread basis is incorrect since the same thread is re-used 
between fixtures in warm-up and runs with concurrency level set to 1.

The change I came up with is to modify the map holding the test node to keep it 
on a per fixture and then per thread basis. The other option would have been to 
run warm-up and single-threaded runs from different threads, but this seemed a 
little bit too complicated since {{AbstractTest}} is sub-classed by many 
benchmarks and would also interfere with issuing halt requests [1].

Can you take a look at the patch, please?

[1] 
https://github.com/apache/jackrabbit-oak/blob/trunk/oak-run/src/main/java/org/apache/jackrabbit/oak/benchmark/AbstractTest.java#L219

> SetPropertyTest benchmark fails on Segment Tar
> ----------------------------------------------
>
>                 Key: OAK-4954
>                 URL: https://issues.apache.org/jira/browse/OAK-4954
>             Project: Jackrabbit Oak
>          Issue Type: Bug
>          Components: segment-tar
>            Reporter: Michael Dürig
>            Assignee: Andrei Dulceanu
>            Priority: Minor
>              Labels: test-failure
>             Fix For: 1.5.18, 1.6
>
>         Attachments: OAK-4954-01.patch, OAK-4954-02.patch
>
>
> The {{SetPropertyTest}} fails on Oak Segment Tar:
> {noformat}
> javax.jcr.InvalidItemStateException: This item 
> [/testfb3e8f1a/ca1ef350-f650-4466-b9e3-7f77d83e6303] does not exist anymore
>       at 
> org.apache.jackrabbit.oak.jcr.delegate.ItemDelegate.checkAlive(ItemDelegate.java:86)
>       at 
> org.apache.jackrabbit.oak.jcr.session.ItemImpl$ItemWriteOperation.checkPreconditions(ItemImpl.java:96)
>       at 
> org.apache.jackrabbit.oak.jcr.session.NodeImpl$35.checkPreconditions(NodeImpl.java:1366)
>       at 
> org.apache.jackrabbit.oak.jcr.delegate.SessionDelegate.prePerform(SessionDelegate.java:615)
>       at 
> org.apache.jackrabbit.oak.jcr.delegate.SessionDelegate.perform(SessionDelegate.java:205)
>       at 
> org.apache.jackrabbit.oak.jcr.session.ItemImpl.perform(ItemImpl.java:112)
>       at 
> org.apache.jackrabbit.oak.jcr.session.NodeImpl.internalSetProperty(NodeImpl.java:1363)
>       at 
> org.apache.jackrabbit.oak.jcr.session.NodeImpl.setProperty(NodeImpl.java:506)
>       at 
> org.apache.jackrabbit.oak.benchmark.SetPropertyTest.runTest(SetPropertyTest.java:65)
>       at 
> org.apache.jackrabbit.oak.benchmark.AbstractTest.execute(AbstractTest.java:372)
>       at 
> org.apache.jackrabbit.oak.benchmark.AbstractTest.runTest(AbstractTest.java:221)
>       at 
> org.apache.jackrabbit.oak.benchmark.AbstractTest.run(AbstractTest.java:197)
>       at 
> org.apache.jackrabbit.oak.benchmark.BenchmarkRunner.main(BenchmarkRunner.java:456)
>       at 
> org.apache.jackrabbit.oak.run.BenchmarkCommand.execute(BenchmarkCommand.java:26)
>       at org.apache.jackrabbit.oak.run.Mode.execute(Mode.java:63)
>       at org.apache.jackrabbit.oak.run.Main.main(Main.java:49)
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to