[
https://issues.apache.org/jira/browse/OAK-4379?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
angela updated OAK-4379:
------------------------
Attachment: OAK-4379.patch
initial draft for adding the ability to save batches of synced users/groups
(limited to {{Delegatee.java}}) and some basic tests.
Note: that patch doesn't yet include the ability to set the batch-size.
Will use this initial draft to run some benchmarks with different batch-sizes;
maybe that gives us a feeling about a reasonable default value and if it really
makes sense to have that configurable.
> Batch mode for SyncMBeanImpl
> ----------------------------
>
> Key: OAK-4379
> URL: https://issues.apache.org/jira/browse/OAK-4379
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: auth-external
> Reporter: angela
> Labels: performance
> Attachments: OAK-4379.patch
>
>
> the {{SyncMBeanImpl}} currently calls {{Session.save()}} for every single
> sync, which IMO make the synchronization methods extra expensive.
> IMHO we should consider introducing a batch mode that reduces the number of
> save calls. the drawback of this was that the complete set of sync-calls
> withing a given batch would succeed or fail. in case of failure the
> 'original' sync-result would need to be replaced by one with operation status
> 'ERR'.
> now that we have the basis for running benchmarks for the {{SyncMBeanImpl}},
> we should be able to verify if this proposal actually has a positive impact
> (though benchmark results from OAK-4119 and OAK-4120 seem to indicate that
> this is the case).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
