Return-Path: X-Original-To: apmail-accumulo-notifications-archive@minotaur.apache.org Delivered-To: apmail-accumulo-notifications-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 71153D8A5 for ; Thu, 23 May 2013 20:35:21 +0000 (UTC) Received: (qmail 87544 invoked by uid 500); 23 May 2013 20:35:21 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 87509 invoked by uid 500); 23 May 2013 20:35:21 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 87500 invoked by uid 99); 23 May 2013 20:35:21 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 23 May 2013 20:35:21 +0000 Date: Thu, 23 May 2013 20:35:21 +0000 (UTC) From: "Hudson (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (ACCUMULO-1393) Removed code related to default ports in client code 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/ACCUMULO-1393?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13665629#comment-13665629 ] Hudson commented on ACCUMULO-1393: ---------------------------------- Integrated in Accumulo-Trunk #887 (See [https://builds.apache.org/job/Accumulo-Trunk/887/]) ACCUMULO-1393 cleaned up unused import warnings from prior changes on this ticket; formatter also seemed to want to get in on the action (Revision 1485778) Result = UNSTABLE ctubbsii : Files : * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperationsImpl.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/ServerServices.java * /accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/lexicoder/BigIntegerLexicoderTest.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/gc/GarbageCollectWriteAheadLogs.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/LiveTServerSet.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/state/TabletStateChangeIterator.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/Admin.java * /accumulo/trunk/test/src/main/java/org/apache/accumulo/test/randomwalk/concurrent/StopTabletServer.java > Removed code related to default ports in client code > ---------------------------------------------------- > > Key: ACCUMULO-1393 > URL: https://issues.apache.org/jira/browse/ACCUMULO-1393 > Project: Accumulo > Issue Type: Improvement > Reporter: Keith Turner > Assignee: Corey J. Nolet > Fix For: 1.6.0 > > Attachments: ACCUMULO-1393.1.patch, ACCUMULO-1393.2.patch > > > The Accumulo client code has code for dealing with default master and tserver ports. This code is old and unused. Its always expected that locations listed in zookeeper and the metadata table will have a ip+port now. If this expectation is not met, it should be treated as an error instead of falling back to a default. Removing this code would clean up the client and logging produced by the client. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira