[jira] [Created] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Thomas Mueller (JIRA)
Thomas Mueller created JCR-3793: --- Summary: vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak Key: JCR-3793 URL: https://issues.apache.org/jira/browse/JCR-3793 Project:

[jira] [Commented] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Jukka Zitting (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14048844#comment-14048844 ] Jukka Zitting commented on JCR-3793: Do we have a good benchmark for this operation. It

[jira] [Commented] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14048847#comment-14048847 ] Thomas Mueller commented on JCR-3793: - [~jukkaz], yes I have a potential patch for Oak,

[jira] [Updated] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Thomas Mueller updated JCR-3793: Attachment: ReorderTest.java Test that shows re-ordering is slow in Oak. vlt: with many child

[jira] [Commented] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Thomas Mueller (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14048852#comment-14048852 ] Thomas Mueller commented on JCR-3793: - Potential patch for Oak (doesn't help much

[jira] [Commented] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Jukka Zitting (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14048973#comment-14048973 ] Jukka Zitting commented on JCR-3793: It looks like we could actually optimize Oak for

[jira] [Commented] (JCR-3793) vlt: with many child nodes, NodeNameList.restoreOrder is very slow with Oak

2014-07-01 Thread Jukka Zitting (JIRA)
[ https://issues.apache.org/jira/browse/JCR-3793?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=14049142#comment-14049142 ] Jukka Zitting commented on JCR-3793: I was able to drop the reordered time from 99306 ms

Re: Some more benchmarks

2014-07-01 Thread Jukka Zitting
Hi, I'm resurrecting this thread with some new findings. I re-ran many of the benchmarks we've been following, pitting Jackrabbit 2.8.0 against Oak 1.0.1 with TarMK. The results look pretty nice: Summary (90%, lower is better) Benchmark Jackrabbit Oak-Tar

Re: svn commit: r1607127 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/segment/SegmentNodeState.java

2014-07-01 Thread Jukka Zitting
Hi, On Tue, Jul 1, 2014 at 12:05 PM, alexparvule...@apache.org wrote: -if (wasCompactedTo(that)) { +if (that.wasCompactedTo(this)) { Nice catch! BR, Jukka Zitting

Re: Some more benchmarks

2014-07-01 Thread Jukka Zitting
Hi, On Tue, Jul 1, 2014 at 9:38 AM, Jukka Zitting jukka.zitt...@gmail.com wrote: I also tried including MongoMK results, but the benchmark got stuck in ConcurrentReadTest. I'll re-try today and will file a bug if I can reproduce the problem. I guess it was a transient problem. Here are the

Re: svn commit: r1607081 - in /jackrabbit/oak/trunk: oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/mk/ oak-commons/src/test/java/org/apache/jackrabbit/oak/commons/mk/ oak-core/src/main/j

2014-07-01 Thread Jukka Zitting
Hi, On Tue, Jul 1, 2014 at 9:57 AM, ste...@apache.org wrote: -return rep.getBlobStore().readBlob(blobId, pos, buff, off, length); +int read = rep.getBlobStore().readBlob(blobId, pos, buff, off, length); +return read 0 ? 0 : read; Shouldn't this (and

Re: svn commit: r1607081 - in /jackrabbit/oak/trunk: oak-commons/src/main/java/org/apache/jackrabbit/oak/commons/mk/ oak-commons/src/test/java/org/apache/jackrabbit/oak/commons/mk/ oak-core/src/main/j

2014-07-01 Thread Jukka Zitting
Hi, On Tue, Jul 1, 2014 at 5:02 PM, Jukka Zitting jukka.zitt...@gmail.com wrote: On Tue, Jul 1, 2014 at 9:57 AM, ste...@apache.org wrote: -return rep.getBlobStore().readBlob(blobId, pos, buff, off, length); +int read = rep.getBlobStore().readBlob(blobId, pos, buff,