Return-Path: X-Original-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-common-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 69E549602 for ; Tue, 27 Mar 2012 04:57:59 +0000 (UTC) Received: (qmail 37957 invoked by uid 500); 27 Mar 2012 04:57:58 -0000 Delivered-To: apmail-hadoop-common-issues-archive@hadoop.apache.org Received: (qmail 37729 invoked by uid 500); 27 Mar 2012 04:57:58 -0000 Mailing-List: contact common-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: common-issues@hadoop.apache.org Delivered-To: mailing list common-issues@hadoop.apache.org Received: (qmail 37643 invoked by uid 99); 27 Mar 2012 04:57:55 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Mar 2012 04:57:55 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 27 Mar 2012 04:57:52 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id B4BDD348E85 for ; Tue, 27 Mar 2012 04:57:31 +0000 (UTC) Date: Tue, 27 Mar 2012 04:57:31 +0000 (UTC) From: "Todd Lipcon (Updated) (JIRA)" To: common-issues@hadoop.apache.org Message-ID: <108217269.21757.1332824251763.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <72388769.21461.1332815246340.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Updated] (HADOOP-8218) RPC.closeProxy shouldn't throw error when closing a mock 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/HADOOP-8218?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Todd Lipcon updated HADOOP-8218: -------------------------------- Attachment: hadoop-8218.txt This much bigger patch implements option #3 as Tom suggested. It will also require a HDFS-side change to fix TestDFSHAAdmin which is currently failing. So, I still think #2 is better, but you both think #3 is better, we'll go ahead with this, and I'll file a new HDFS JIRA to fix TestDFSHAAdmin to use the new mock util once this is committed. > RPC.closeProxy shouldn't throw error when closing a mock > -------------------------------------------------------- > > Key: HADOOP-8218 > URL: https://issues.apache.org/jira/browse/HADOOP-8218 > Project: Hadoop Common > Issue Type: Bug > Components: ipc, test > Affects Versions: 0.24.0 > Reporter: Todd Lipcon > Assignee: Todd Lipcon > Priority: Critical > Attachments: hadoop-8218.txt, hadoop-8218.txt > > > HADOOP-8202 changed the behavior of RPC.stopProxy() to throw an exception if called on an object which doesn't implement Closeable. Unfortunately, we use mock objects in many test cases, and those mocks don't implement Closeable. This is causing TestZKFailoverController to fail in trunk, for example. -- 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