Return-Path: X-Original-To: apmail-hbase-issues-archive@www.apache.org Delivered-To: apmail-hbase-issues-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 1323ED53C for ; Sat, 7 Jul 2012 21:32:37 +0000 (UTC) Received: (qmail 5809 invoked by uid 500); 7 Jul 2012 21:32:36 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 5701 invoked by uid 500); 7 Jul 2012 21:32:36 -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 5688 invoked by uid 99); 7 Jul 2012 21:32:36 -0000 Received: from issues-vm.apache.org (HELO issues-vm) (140.211.11.160) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 07 Jul 2012 21:32:36 +0000 Received: from isssues-vm.apache.org (localhost [127.0.0.1]) by issues-vm (Postfix) with ESMTP id 67387142854 for ; Sat, 7 Jul 2012 21:32:36 +0000 (UTC) Date: Sat, 7 Jul 2012 21:32:36 +0000 (UTC) From: "Himanshu Vashishtha (JIRA)" To: issues@hbase.apache.org Message-ID: <1472034978.19319.1341696756424.JavaMail.jiratomcat@issues-vm> In-Reply-To: <1279073491.43210.1331295657808.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HBASE-5549) Master can fail if ZooKeeper session expires MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HBASE-5549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13408782#comment-13408782 ] Himanshu Vashishtha commented on HBASE-5549: -------------------------------------------- Thanks for looking into this. Since the update is happening inside an inner class, it should be final (volatile is not possible). I will wait/notify on isClosed array object, and upload the patch in a new jira. > Master can fail if ZooKeeper session expires > -------------------------------------------- > > Key: HBASE-5549 > URL: https://issues.apache.org/jira/browse/HBASE-5549 > Project: HBase > Issue Type: Bug > Components: master, zookeeper > Affects Versions: 0.96.0 > Environment: all > Reporter: nkeywal > Assignee: nkeywal > Priority: Minor > Fix For: 0.96.0 > > Attachments: 5549.v10.patch, 5549.v11.patch, 5549.v6.patch, 5549.v7.patch, 5549.v8.patch, 5549.v9.patch, nochange.patch > > > There is a retry mechanism in RecoverableZooKeeper, but when the session expires, the whole ZooKeeperWatcher is recreated, hence the retry mechanism does not work in this case. This is why a sleep is needed in TestZooKeeper#testMasterSessionExpired: we need to wait for ZooKeeperWatcher to be recreated before using the connection. > This can happen in real life, it can happen when: > - master & zookeeper starts > - zookeeper connection is cut > - master enters the retry loop > - in the meantime the session expires > - the network comes back, the session is recreated > - the retries continues, but on the wrong object, hence fails. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira