[ 
https://issues.apache.org/jira/browse/OAK-6177?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16000454#comment-16000454
 ] 

angela commented on OAK-6177:
-----------------------------

[~alex.parvulescu], i would suggest to cover that with a separate benchmark... 
as far as i remember adding an existing member used to be almost as expensive 
as adding a new one... so, i would like to keep that as part of the benchmarks 
to make sure we would be able to catch any regression in that area. Adding a 
new "AddUniqueMembersTest" that extends from "AddMembersTest" and just provides 
different 'ids' should be doable with moderate effort, shouldn't it? Also we 
can compare figures between the 2 as we go ahead.

> AddMembersTest should add unique items to groups
> ------------------------------------------------
>
>                 Key: OAK-6177
>                 URL: https://issues.apache.org/jira/browse/OAK-6177
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: benchmarks
>            Reporter: Alex Parvulescu
>            Assignee: Alex Parvulescu
>            Priority: Minor
>         Attachments: OAK-6177.patch
>
>
> In the current setup the {{AddMembersTest}} benchmark exhausts unique user 
> ids and quite quickly starts adding existing items to the same groups 
> reducing the usefulness of the profiling results (the Group code bails early 
> for existing members).
> I think we should make this benchmark add only unique items. [~anchela] what 
> do you think?



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to