Return-Path: X-Original-To: apmail-curator-dev-archive@minotaur.apache.org Delivered-To: apmail-curator-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id C18AB17DBF for ; Wed, 20 May 2015 00:17:46 +0000 (UTC) Received: (qmail 39342 invoked by uid 500); 20 May 2015 00:17:46 -0000 Delivered-To: apmail-curator-dev-archive@curator.apache.org Received: (qmail 39287 invoked by uid 500); 20 May 2015 00:17:46 -0000 Mailing-List: contact dev-help@curator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@curator.apache.org Delivered-To: mailing list dev@curator.apache.org Received: (qmail 39266 invoked by uid 99); 20 May 2015 00:17:45 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 20 May 2015 00:17:45 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 15AB7E35A4; Wed, 20 May 2015 00:17:45 +0000 (UTC) From: cammckenzie To: dev@curator.apache.org Reply-To: dev@curator.apache.org References: In-Reply-To: Subject: [GitHub] curator pull request: [CURATOR-217] Use new Watcher Removal APIs i... Content-Type: text/plain Message-Id: <20150520001745.15AB7E35A4@git1-us-west.apache.org> Date: Wed, 20 May 2015 00:17:45 +0000 (UTC) Github user cammckenzie commented on the pull request: https://github.com/apache/curator/pull/82#issuecomment-103703152 Other than my minor comments, this looks good to me. Still needs to have the updates to CURATOR-161 merged into it also as currently one of the CURATOR-161 related unit tests is breaking. Are there any other recipes that require watcher removal? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastructure@apache.org or file a JIRA ticket with INFRA. ---