[ 
https://issues.apache.org/jira/browse/ACCUMULO-259?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13562514#comment-13562514
 ] 

Hudson commented on ACCUMULO-259:
---------------------------------

Integrated in Accumulo-Trunk #668 (See 
[https://builds.apache.org/job/Accumulo-Trunk/668/])
    ACCUMULO-259 - Merged into trunk. May FSM have mercy on us all (Revision 
1438354)

     Result = FAILURE
vines : 
Files : 
* /accumulo/trunk
* /accumulo/trunk/assemble
* /accumulo/trunk/bin/accumulo
* /accumulo/trunk/conf/examples/1GB/native-standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/1GB/standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/2GB/native-standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/2GB/standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/3GB/native-standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/3GB/standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/512MB/native-standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/512MB/standalone/accumulo-site.xml
* /accumulo/trunk/conf/examples/vfs-classloader/accumulo-site.xml
* /accumulo/trunk/core
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/cli/ClientOnDefaultTable.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/cli/ClientOnRequiredTable.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/cli/ClientOpts.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/AccumuloSecurityException.java
* 
/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/admin/SecurityOperations.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/SecurityOperationsImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/admin/TableOperationsImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/BatchWriterImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/ConnectorImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/MetadataLocationObtainer.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/MultiTableBatchWriterImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/OfflineScanner.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/ScannerImpl.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/ScannerIterator.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletLocator.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchDeleter.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchReader.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchReaderIterator.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/ThriftScanner.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/impl/Writer.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/mapred/AccumuloOutputFormat.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapred/InputFormatBase.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormat.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/InputFormatBase.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/ConfiguratorBase.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mapreduce/lib/util/InputConfigurator.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mock/MockInstance.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/client/mock/MockSecurityOperations.java
* /accumulo/trunk/core/src/main/java/org/apache/accumulo/core/conf/Property.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/master/thrift/MasterClientService.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/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/tokens
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/AccumuloToken.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/InstanceTokenWrapper.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/KerberosToken.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/LoginCallbackHandler.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/PasswordUpdatable.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/TokenHelper.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/security/tokens/UserPassToken.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/util/MetadataTable.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/Shell.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/CreateUserCommand.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/PasswdCommand.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/TraceCommand.java
* 
/accumulo/trunk/core/src/main/java/org/apache/accumulo/core/util/shell/commands/UserCommand.java
* /accumulo/trunk/core/src/main/thrift/client.thrift
* /accumulo/trunk/core/src/main/thrift/gc.thrift
* /accumulo/trunk/core/src/main/thrift/master.thrift
* /accumulo/trunk/core/src/main/thrift/security.thrift
* /accumulo/trunk/core/src/main/thrift/tabletserver.thrift
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/cli/TestClientOpts.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/admin/FindMaxTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/impl/TabletLocatorImplTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mapreduce/AccumuloFileOutputFormatTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mapreduce/AccumuloInputFormatTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mapreduce/AccumuloOutputFormatTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mapreduce/AccumuloRowInputFormatTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mock/MockConnectorTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mock/MockTableOperationsTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/client/mock/TestBatchScanner821.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/iterators/user/IntersectingIteratorTest.java
* 
/accumulo/trunk/core/src/test/java/org/apache/accumulo/core/iterators/user/RowFilterTest.java
* /accumulo/trunk/examples
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/client/RandomBatchScanner.java
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/client/ReadWriteExample.java
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/dirlist/Viewer.java
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/filedata/FileDataIngest.java
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/filedata/FileDataQuery.java
* 
/accumulo/trunk/examples/simple/src/main/java/org/apache/accumulo/examples/simple/shard/ContinuousQuery.java
* 
/accumulo/trunk/examples/simple/src/test/java/org/apache/accumulo/examples/simple/dirlist/CountTest.java
* 
/accumulo/trunk/examples/simple/src/test/java/org/apache/accumulo/examples/simple/filedata/ChunkInputFormatTest.java
* 
/accumulo/trunk/examples/simple/src/test/java/org/apache/accumulo/examples/simple/filedata/ChunkInputStreamTest.java
* /accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/ZooStore.java
* 
/accumulo/trunk/fate/src/main/java/org/apache/accumulo/fate/zookeeper/ZooSession.java
* /accumulo/trunk/packages
* /accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
* 
/accumulo/trunk/proxy/src/main/java/org/apache/accumulo/proxy/thrift/IteratorSetting.java
* /accumulo/trunk/server
* /accumulo/trunk/server/pom.xml
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/client/BulkImporter.java
* 
/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/gc/GarbageCollectWriteAheadLogs.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/gc/SimpleGarbageCollector.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/Master.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/state/MetaDataStateStore.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/state/MetaDataTableScanner.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/state/RootTabletStateStore.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/state/SetGoalState.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/tableOps/BulkImport.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/tableOps/CloneTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/tableOps/CreateTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/tableOps/DeleteTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/master/tableOps/ImportTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/metanalysis/IndexMeta.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/monitor/Monitor.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/monitor/servlets/TServersServlet.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/AuditedSecurityOperation.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/Auditor.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/Authenticator.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/SecurityConstants.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/SecurityOperation.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/SecurityUtil.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/ZKAuthenticator.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/Authenticator.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/Authorizor.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/InsecureAuthenticator.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/InsecurePermHandler.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/PermissionHandler.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/ZKAuthenticator.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/ZKAuthorizor.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/ZKPermHandler.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/security/handler/ZKSecurityTool.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/tabletserver/Tablet.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/tabletserver/TabletServer.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/tabletserver/mastermessage/MasterMessage.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/tabletserver/mastermessage/SplitReportMessage.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/tabletserver/mastermessage/TabletStatusMessage.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/GetMasterStats.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/QueryMetadataTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/WrongTabletTest.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/continuous/ContinuousStatsCollector.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/FunctionalTest.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/PermissionsTest.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/VisibilityTest.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/functional/ZombieTServer.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/performance/scan/CollectTabletStats.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/performance/thrift/NullTserver.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/Module.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/State.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/concurrent/CreateUser.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/image/Commit.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/image/ImageFixture.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/image/Write.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/multitable/Commit.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/multitable/CopyTool.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/multitable/Write.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/AlterSystemPerm.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/AlterTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/AlterTablePerm.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/Authenticate.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/ChangePass.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/CreateTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/CreateUser.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/DropTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/DropUser.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityFixture.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SecurityHelper.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/SetAuths.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/TableOp.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/Validate.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/security/WalkingSecurity.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/sequential/BatchVerify.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/sequential/Commit.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/sequential/MapRedVerifyTool.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/test/randomwalk/sequential/Write.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/trace/TraceServer.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/Admin.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/CheckForMetadataProblems.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/FindOfflineTablets.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/Initialize.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/MetadataTable.java
* 
/accumulo/trunk/server/src/main/java/org/apache/accumulo/server/util/VerifyTabletAssignments.java
* 
/accumulo/trunk/server/src/test/java/org/apache/accumulo/server/gc/TestConfirmDeletes.java
* 
/accumulo/trunk/server/src/test/java/org/apache/accumulo/server/master/TestMergeState.java
* 
/accumulo/trunk/server/src/test/java/org/apache/accumulo/server/security/ZKAuthenticatorTest.java
* 
/accumulo/trunk/server/src/test/java/org/apache/accumulo/server/security/handler
* 
/accumulo/trunk/server/src/test/java/org/apache/accumulo/server/security/handler/ZKAuthenticatorTest.java
* /accumulo/trunk/src
* 
/accumulo/trunk/test/src/main/java/org/apache/accumulo/test/MiniAccumuloCluster.java
* 
/accumulo/trunk/test/src/test/java/org/apache/accumulo/test/MiniAccumuloClusterTest.java
* /accumulo/trunk/test/system/auto/TestUtils.py
* /accumulo/trunk/test/system/randomwalk/conf/modules/Security.xml
* 
/accumulo/trunk/trace/src/main/java/org/apache/accumulo/cloudtrace/thrift/RemoteSpan.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
>
>
> 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

Reply via email to