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 6F9DE9544 for ; Wed, 6 Mar 2013 20:52:14 +0000 (UTC) Received: (qmail 36245 invoked by uid 500); 6 Mar 2013 20:52:14 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 36188 invoked by uid 500); 6 Mar 2013 20:52:14 -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 36132 invoked by uid 99); 6 Mar 2013 20:52:14 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 06 Mar 2013 20:52:13 +0000 Date: Wed, 6 Mar 2013 20:52:13 +0000 (UTC) From: "Hudson (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (ACCUMULO-259) ZKAuthentictor needs to be split up 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-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13595098#comment-13595098 ] Hudson commented on ACCUMULO-259: --------------------------------- Integrated in Accumulo-Trunk-Hadoop-2.0 #120 (See [https://builds.apache.org/job/Accumulo-Trunk-Hadoop-2.0/120/]) ACCUMULO-259 Changing the thrift getAuthentictorClass from a master operation to a tserver one (Revision 1453530) Result = SUCCESS vines : Files : * /accumulo/trunk * /accumulo/trunk/core * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/Instance.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/ZooKeeperInstance.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperationsImpl.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ClientService.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ConfigurationType.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/SecurityErrorCode.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/TableOperation.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/TableOperationExceptionType.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ThriftSecurityException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ThriftTableOperationException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/thrift/ThriftTest.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/InitialMultiScan.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/InitialScan.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/IterInfo.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/MapFileInfo.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/MultiScanResult.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/ScanResult.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TColumn.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TConstraintViolationSummary.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TKey.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TKeyExtent.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TKeyValue.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TMutation.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/TRange.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/data/thrift/UpdateErrors.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/gc/thrift/GCMonitorService.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/gc/thrift/GCStatus.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/gc/thrift/GcCycleStats.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/Compacting.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/DeadServer.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/MasterClientService.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/MasterGoalState.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/MasterMonitorInfo.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/MasterState.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/RecoveryException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/RecoveryStatus.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/TableInfo.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/TableOperation.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/TabletLoadState.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/TabletServerStatus.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/master/thrift/TabletSplit.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/thrift/AuthInfo.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/thrift/SecurityErrorCode.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/thrift/TCredentials.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/thrift/ThriftSecurityException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ActionStats.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ActiveCompaction.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ActiveScan.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/CompactionReason.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/CompactionType.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ConstraintViolationException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/IteratorConfig.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/NoSuchScanIDException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/NotServingTabletException.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ScanState.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/ScanType.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TIteratorSetting.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TabletClientService.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TabletStats.java * /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/tabletserver/thrift/TooManyFilesException.java * /accumulo/trunk/core/src/main/thrift/client.thrift * /accumulo/trunk/core/src/main/thrift/master.thrift * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/AccumuloException.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/AccumuloProxy.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/AccumuloSecurityException.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ActiveCompaction.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ActiveScan.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/BatchScanOptions.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/Column.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ColumnUpdate.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/CompactionReason.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/CompactionType.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/IteratorScope.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/IteratorSetting.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/Key.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/KeyExtent.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/KeyValue.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/KeyValueAndPeek.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/NoMoreEntriesException.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/PartialKey.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/Range.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ScanColumn.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ScanOptions.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ScanResult.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ScanState.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/ScanType.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/SystemPermission.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/TableExistsException.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/TableNotFoundException.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/TablePermission.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/TimeType.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/UnknownScanner.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/UnknownWriter.java * /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/WriterOptions.java * /accumulo/trunk/server * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/client/ClientServiceHandler.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/client/HdfsZooInstance.java * /accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/Master.java * /accumulo/trunk/start/src/main/java/org/apache/accumulo/start/Main.java * /accumulo/trunk/trace/src/main/java/org/apache/accumulo/trace/thrift/RemoteSpan.java * /accumulo/trunk/trace/src/main/java/org/apache/accumulo/trace/thrift/SpanReceiver.java * /accumulo/trunk/trace/src/main/java/org/apache/accumulo/trace/thrift/TInfo.java * /accumulo/trunk/trace/src/main/java/org/apache/accumulo/trace/thrift/TestService.java > ZKAuthentictor needs to be split up > ----------------------------------- > > Key: ACCUMULO-259 > URL: https://issues.apache.org/jira/browse/ACCUMULO-259 > Project: Accumulo > Issue Type: Improvement > Components: master, tserver > Reporter: John Vines > Assignee: John Vines > Fix For: 1.5.0 > > Attachments: accumulo259_explained.pdf > > > Currently the ZKAuthenticator does both the checking of credentials as well as the functionality of the Authenticator. We need to split those into two different classes for improved testing as well as pluggability of the security scheme. -- 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