Return-Path: Delivered-To: apmail-hbase-issues-archive@www.apache.org Received: (qmail 68736 invoked from network); 17 Dec 2010 06:13:07 -0000 Received: from unknown (HELO mail.apache.org) (140.211.11.3) by 140.211.11.9 with SMTP; 17 Dec 2010 06:13:07 -0000 Received: (qmail 26032 invoked by uid 500); 17 Dec 2010 06:06:26 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 25856 invoked by uid 500); 17 Dec 2010 06:06:25 -0000 Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list issues@hbase.apache.org Received: (qmail 25426 invoked by uid 99); 17 Dec 2010 06:06:25 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 17 Dec 2010 06:06:25 +0000 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.22] (HELO thor.apache.org) (140.211.11.22) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 17 Dec 2010 06:06:22 +0000 Received: from thor (localhost [127.0.0.1]) by thor.apache.org (8.13.8+Sun/8.13.8) with ESMTP id oBH661RX021733 for ; Fri, 17 Dec 2010 06:06:01 GMT Message-ID: <28894814.172601292565961064.JavaMail.jira@thor> Date: Fri, 17 Dec 2010 01:06:01 -0500 (EST) From: "stack (JIRA)" To: issues@hbase.apache.org Subject: [jira] Commented: (HBASE-3366) WALObservers should be notified before the lock In-Reply-To: <2329869.150051292463541629.JavaMail.jira@thor> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 X-Virus-Checked: Checked by ClamAV on apache.org [ https://issues.apache.org/jira/browse/HBASE-3366?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12972364#action_12972364 ] stack commented on HBASE-3366: ------------------------------ You are saying that the newly rolled file, the one that is created inside the synchronized block, that it got some edits and these edits were not "observed" because notification of roll happened after the new file was created? If so, +1 on your patch. > WALObservers should be notified before the lock > ----------------------------------------------- > > Key: HBASE-3366 > URL: https://issues.apache.org/jira/browse/HBASE-3366 > Project: HBase > Issue Type: Bug > Reporter: Jean-Daniel Cryans > Assignee: Jean-Daniel Cryans > Fix For: 0.90.0 > > Attachments: HBASE-3366.patch > > > TestReplication failed on the last 0.90 build because the log of the machine that was killed was rolled and got a few edits in before it was put in a ZK queue. > This fix is pretty easy, the notification should be sent before the lock is acquired instead of after that. It shouldn't have been like that since the beginning but I guess I missed it when I narrowed the lock. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.