Return-Path: Delivered-To: apmail-jackrabbit-dev-archive@www.apache.org Received: (qmail 16057 invoked from network); 5 Mar 2009 11:56:20 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 5 Mar 2009 11:56:20 -0000 Received: (qmail 20998 invoked by uid 500); 5 Mar 2009 11:56:19 -0000 Delivered-To: apmail-jackrabbit-dev-archive@jackrabbit.apache.org Received: (qmail 20982 invoked by uid 500); 5 Mar 2009 11:56:19 -0000 Mailing-List: contact dev-help@jackrabbit.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@jackrabbit.apache.org Delivered-To: mailing list dev@jackrabbit.apache.org Received: (qmail 20973 invoked by uid 99); 5 Mar 2009 11:56:18 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 05 Mar 2009 03:56:18 -0800 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.140] (HELO brutus.apache.org) (140.211.11.140) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 05 Mar 2009 11:56:18 +0000 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id 30DDF234C48D for ; Thu, 5 Mar 2009 03:55:56 -0800 (PST) Message-ID: <74463782.1236254156185.JavaMail.jira@brutus> Date: Thu, 5 Mar 2009 03:55:56 -0800 (PST) From: "Jukka Zitting (JIRA)" To: dev@jackrabbit.apache.org Subject: [jira] Updated: (JCR-2000) Deadlock on concurrent commits In-Reply-To: <1999483538.1235576941753.JavaMail.jira@brutus> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/JCR-2000?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jukka Zitting updated JCR-2000: ------------------------------- Attachment: search-on-sism.patch The culprit seems to be the system search manager, that (like LockManager above) uses the SessionItemStateManager of the system session for looking up items. The attached patch (search-on-sism.patch) makes the system search manager use the underlying SharedItemStateManager instance instead of the SessionItemStateManager. > Deadlock on concurrent commits > ------------------------------ > > Key: JCR-2000 > URL: https://issues.apache.org/jira/browse/JCR-2000 > Project: Jackrabbit Content Repository > Issue Type: Bug > Components: jackrabbit-core, transactions > Affects Versions: 1.5.3 > Reporter: Jukka Zitting > Assignee: Jukka Zitting > Fix For: 1.5.4 > > Attachments: JCR-2000.patch, JCR-2000.patch, search-on-sism.patch, thread-join.patch > > > As reported in the followup to JCR-1979, there's a case where two transactions may be concurrently inside a commit. This is bad as it breaks the main assumption in http://jackrabbit.apache.org/concurrency-control.html about all transactions first acquiring the versioning write lock. > Looking deeper into this I find that the versioning write lock is only acquired if the transaction being committed contains versioning operations. This is incorrect as all transactions in any case need to access the version store when checking for references. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.