From hdfs-issues-return-32831-apmail-hadoop-hdfs-issues-archive=hadoop.apache.org@hadoop.apache.org Wed Jan 25 19:50:05 2012 Return-Path: X-Original-To: apmail-hadoop-hdfs-issues-archive@minotaur.apache.org Delivered-To: apmail-hadoop-hdfs-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 A7CBD9D52 for ; Wed, 25 Jan 2012 19:50:05 +0000 (UTC) Received: (qmail 49371 invoked by uid 500); 25 Jan 2012 19:50:05 -0000 Delivered-To: apmail-hadoop-hdfs-issues-archive@hadoop.apache.org Received: (qmail 49017 invoked by uid 500); 25 Jan 2012 19:50:04 -0000 Mailing-List: contact hdfs-issues-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: hdfs-issues@hadoop.apache.org Delivered-To: mailing list hdfs-issues@hadoop.apache.org Received: (qmail 48980 invoked by uid 99); 25 Jan 2012 19:50:03 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 25 Jan 2012 19:50:03 +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; Wed, 25 Jan 2012 19:50:01 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id 7E037163A0F for ; Wed, 25 Jan 2012 19:49:40 +0000 (UTC) Date: Wed, 25 Jan 2012 19:49:40 +0000 (UTC) From: "Aaron T. Myers (Commented) (JIRA)" To: hdfs-issues@hadoop.apache.org Message-ID: <1258025199.77713.1327520980517.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <289174661.54206.1326926319881.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HDFS-2805) HA: Add a test for a federated cluster with HA NNs 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/HDFS-2805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13193252#comment-13193252 ] Aaron T. Myers commented on HDFS-2805: -------------------------------------- Thanks a lot for providing a patch for this issue, Brandon. A few comments: # simpleHAFederatedTopology - Perhaps the name service and NN numbering should start at 0, instead of 1? There's no reason it can't start at 0, and I'd prefer to be as consistent as possible. (Most of the HA tests start at 0, not 1.) # configureFailoverFs - should use javadoc-style method comment with @param tags. # configureFailoverFs - rather than duplicating the code in the configuredFailoverFs which doesn't take a nameservice index, you should implement the other in terms of the one which does take a name service index. # setFailoverConfigurations - you added another parameter without updating all callers of the method, so TestBalancerWithHANameNodes no longer compiles. You should either update the caller or add another HATestUtil#setFailoverConfigurations method which defaults to the first name service. I probably prefer the latter. # MiniDFSCluster#createNameNodesAndSetConf - since you moved the aggregation of the list of nameservice IDs higher in the method, there's no need for the nsIds list lower in the method. # testManualFOFB - I don't like that the log messages will say "failing over to NN 3", etc when we're testing the failover of a higher name service ID. I'd much prefer "failing over to NN 1 in name service 2", for example. # testManualFOFB - no need to abbreviate this method name, and it's not at all obvious what it stands for, especially since the method comment just says "test manual fofb" # testManualFOFB - pleause use a javadoc-style method comment, and put a blank line between methods in a class. # testManualFOFB - please put spaces around arithmetic operators "+" and "*". > HA: Add a test for a federated cluster with HA NNs > -------------------------------------------------- > > Key: HDFS-2805 > URL: https://issues.apache.org/jira/browse/HDFS-2805 > Project: Hadoop HDFS > Issue Type: Sub-task > Components: ha, name-node > Affects Versions: HA branch (HDFS-1623) > Reporter: Aaron T. Myers > Assignee: Brandon Li > Attachments: HDFS-2805.3.txt, HDFS-2805.second.txt, HDFS-2805.txt > > > Add a test for configuring/interacting with a federated cluster wherein each name service is itself HA. -- 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