[
https://issues.apache.org/jira/browse/OAK-585?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13566306#comment-13566306
]
Mete Atamel edited comment on OAK-585 at 1/30/13 9:00 AM:
----------------------------------------------------------
BTW, it looks to me that ConflictingCommitException should be renamed to
ConcurrentCommitException instead because it's thrown whenever there's a
concurrent but not necessarily conflicting commits. Instead,
readAndMergeExistingNodes should be throwing ConflictingCommitException and
CommitCommand should only retry in ConcurrentCommitException.
was (Author: meteatamel):
BTW, it looks to me that ConflictingCommitException should be renamed to
ConcurrentCommitException instead because it's thrown whenever there's a
concurrent but not necessarily conflicting commits. Instead,
readAndMergeExistingNodes should be throwing ConflictingCommitException.
> Reduce cost of commit retry
> ---------------------------
>
> Key: OAK-585
> URL: https://issues.apache.org/jira/browse/OAK-585
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Components: mongomk
> Affects Versions: 0.6
> Reporter: Marcel Reutegger
> Attachments: OAK-585-1&2.patch, OAK-585-1&2.patch, OAK-585-1.patch,
> OAK-585-2.patch, OAK-585-3.patch
>
>
> MongoMK currently has a retry mechanism when it commits changes. Its main
> purpose is to make sure the headRevId in the sync collection is updated
> consistently and concurrent conflicting commits are detected. Under higher
> concurrency it may happen that commits are retried multiple times, even when
> commits are non-conflicting. In this case its only about consistently
> updating the headRevId. When this happens the commit will get a new revision
> and the nodes of the commit are written again with the new revision, even
> though no changes to the nodes are needed. Only the revision needed an update.
> The cost of a retry should be reduced in this case. At least a commit retry
> should not leave garbage in the nodes and commits collections.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira