[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-21 Thread Ted Yu (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16261937#comment-16261937
 ] 

Ted Yu commented on HBASE-19200:


First, TestAcidGuarantees hung.
With workaround, TestAdmin2.testCheckHBaseAvailableWithoutCluster hangs.

I think this should be reverted. Once CURATOR-443 is fixed and we upgrade to 
next curator release, this along with proper fix for the (currently) hanging 
tests can go in again.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-20 Thread Ted Yu (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16260211#comment-16260211
 ] 

Ted Yu commented on HBASE-19200:


I ran TestAcidGuarantees with commit 31234eb862fdd7ee4917a2f74001182565ffbfd9 :
{code}
[INFO] Tests run: 18, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 710.702 
s - in org.apache.hadoop.hbase.TestAcidGuarantees
{code}
With this change:
{code}
[INFO] Apache HBase - Server .. FAILURE [15:07 min]
{code}
Same machine, Linux 3.10.0-327.28.3.el7.x86_64 #1 SMP Thu Aug 18 19:05:49 UTC 
2016 x86_64 x86_64 x86_64 GNU/Linux
{code}
Maven home: /apache-maven-3.5.0
Java version: 1.8.0_131, vendor: Oracle Corporation
{code}
[~chia7712] first discovered what caused TestAcidGuarantees to hang.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-14 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16252126#comment-16252126
 ] 

Appy commented on HBASE-19200:
--

Sg [~stack]. Looks like Duo added a release note. Let me do minor rewording.


> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-13 Thread stack (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16250998#comment-16250998
 ] 

stack commented on HBASE-19200:
---

[~appy] ok. Sounds good. i took a look at the zk break-out patch and i see that 
it is zk util, watchers, server-side util, minizkcluster, that are in 
hbase-zookeeper, stuff the hbase-client can do w/o.

We need release note here though? Overview on the nice cleanup?

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-13 Thread Sean Busbey (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16250509#comment-16250509
 ] 

Sean Busbey commented on HBASE-19200:
-

please see HBASE-19248. I'll see if I can fix it shortly, but if not I'm 
planning to revert this change tomorrow.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-13 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16250052#comment-16250052
 ] 

Appy commented on HBASE-19200:
--

[~stack], after this patch, hbase-client doesn't depend on anything except 
ZKMetaData and ZNodePath. Now we can move most stuff out in separate 
hbase-zookeeper module.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-11 Thread stack (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16248612#comment-16248612
 ] 

stack commented on HBASE-19200:
---

This is great. I love that we move ahead on the old Duo plan of untangling 
client and zk.

I am a little unclear though on what exactly was delivered. hbase-client will 
still depend on the new hbase-zookeeper module because of ZKLeaderManager, et 
al? Can we have release note that states what we get here? Thanks lads.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-11 Thread Duo Zhang (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16248499#comment-16248499
 ] 

Duo Zhang commented on HBASE-19200:
---

Thanks [~appy] for helping committing this.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Hudson (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16248280#comment-16248280
 ] 

Hudson commented on HBASE-19200:


FAILURE: Integrated in Jenkins build HBase-2.0 #832 (See 
[https://builds.apache.org/job/HBase-2.0/832/])
HBASE-19200 Make hbase-client only depend on ZKAsyncRegistry and (appy: rev 
30f55f23166f3b0652bb2a96f8bbb1b2bcde4ace)
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/MetaCache.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestClientNoCluster.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZkSplitLogWorkerCoordination.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/ZNodeClearer.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/backup/example/ZKTableArchiveClient.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegistryFactory.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureCoordinator.java
* (edit) 
hbase-it/src/test/java/org/apache/hadoop/hbase/test/IntegrationTestZKAndFSPermissions.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMetaShutdownHandler.java
* (add) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKMetadata.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestReplicationTrackerZKImpl.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZooKeeperNodeTracker.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZooKeeperKeepAliveConnection.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/SplitOrMergeTracker.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/ZKNamespaceManager.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZKSplitLogManagerCoordination.java
* (edit) 
hbase-rsgroup/src/main/java/org/apache/hadoop/hbase/rsgroup/RSGroupInfoManagerImpl.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/backup/example/HFileArchiveManager.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/ZKPermissionWatcher.java
* (edit) 
hbase-rsgroup/src/test/java/org/apache/hadoop/hbase/rsgroup/VerifyingRSGroupAdminClient.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/GenericTestUtils.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZooKeeperRegistry.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestSplitLogWorker.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/AuthenticationTokenSecretManager.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureMemberRpcs.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKSplitLog.java
* (edit) 
hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ReplicationPeersZKImpl.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestMasterReplication.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/util/ZKDataMigrator.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/ZKSecretWatcher.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/Registry.java
* (edit) 
hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ReplicationQueuesZKImpl.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestTableStateManager.java
* (edit) 
hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ReplicationQueuesClientZKImpl.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ZKVisibilityLabelWatcher.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClusterConnection.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestBufferedMutator.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZkAclReset.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/RecoverableZooKeeper.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZKAsyncRegistry.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestReplicationStateZKImpl.java
* (edit)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Hudson (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16248215#comment-16248215
 ] 

Hudson commented on HBASE-19200:


FAILURE: Integrated in Jenkins build HBase-Trunk_matrix #4032 (See 
[https://builds.apache.org/job/HBase-Trunk_matrix/4032/])
HBASE-19200 Make hbase-client only depend on ZKAsyncRegistry and (appy: rev 
72270866cbacbcfd10f2ad2cce33362577eeef4d)
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/AuthenticationTokenSecretManager.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZKMulti.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/ZKPermissionWatcher.java
* (edit) 
hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ReplicationPeersZKImpl.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/backup/example/ZKTableArchiveClient.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestAsyncProcess.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/security/token/TestZKSecretWatcherRefreshKeys.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZooKeeperNodeTracker.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/MasterAddressTracker.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestSplitLogWorker.java
* (edit) 
hbase-it/src/test/java/org/apache/hadoop/hbase/IntegrationTestMetaReplicas.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ClusterConnection.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/SplitOrMergeTracker.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/backup/example/HFileArchiveManager.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZKAsyncRegistry.java
* (edit) hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestAssignmentListener.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/regionserver/TestReplicationSourceManager.java
* (edit) 
hbase-rsgroup/src/test/java/org/apache/hadoop/hbase/rsgroup/VerifyingRSGroupAdminClient.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestReplicationTrackerZKImpl.java
* (edit) 
hbase-it/src/test/java/org/apache/hadoop/hbase/test/IntegrationTestZKAndFSPermissions.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/MetaCache.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZooKeeperKeepAliveConnection.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/RegistryFactory.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZkAclReset.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestClientNoCluster.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestReplicationStateHBaseImpl.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/coordination/ZkSplitLogWorkerCoordination.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKSplitLog.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/HBaseAdmin.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/ActiveMasterManager.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/token/ZKSecretWatcher.java
* (delete) 
hbase-client/src/main/java/org/apache/hadoop/hbase/client/ZooKeeperRegistry.java
* (add) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKMetadata.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/ZKNamespaceManager.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZooKeeperWatcher.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/client/TestMetaWithReplicas.java
* (add) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/DoNothingAsyncRegistry.java
* (edit) 
hbase-server/src/test/java/org/apache/hadoop/hbase/replication/TestMasterReplication.java
* (edit) 
hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZooKeeperWatcher.java
* (edit) 
hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ReplicationQueuesClientZKImpl.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/security/visibility/ZKVisibilityLabelWatcher.java
* (edit) 
hbase-client/src/test/java/org/apache/hadoop/hbase/client/TestBufferedMutator.java
* (edit) 
hbase-server/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureCoordinator.java

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247868#comment-16247868
 ] 

Appy commented on HBASE-19200:
--

Pushed to master and branch-2.
Thanks for cleaning it up sooner just to unblock me [~Apache9].

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247861#comment-16247861
 ] 

Appy commented on HBASE-19200:
--

It's 2am Duo's time. Let me commit it so that i can also start progressing 
HBASE-19114. I hope he won't mind.
If he does - sorry, and you can have my desserts in next HBaseCon :)

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200-v5.patch, 
> HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Hadoop QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247816#comment-16247816
 ] 

Hadoop QA commented on HBASE-19200:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
16s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Findbugs executables are not available. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 23 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
10s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  3m 
58s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
37s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  1m 
57s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  6m 
 7s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
13s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
11s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
 5s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
37s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m 
37s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
25s{color} | {color:red} hbase-client: The patch generated 3 new + 339 
unchanged - 30 fixed = 342 total (was 369) {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
58s{color} | {color:red} hbase-server: The patch generated 1 new + 587 
unchanged - 1 fixed = 588 total (was 588) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
 5s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
44m 40s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 2.7.4 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m  
9s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
38s{color} | {color:green} hbase-client in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
10s{color} | {color:green} hbase-replication in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}102m 
26s{color} | {color:green} hbase-server in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m  
4s{color} | {color:green} hbase-rsgroup in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
26s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  0m 
57s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}175m 17s{color} | 
{color:black} {color} |
\\
\\
|| Subsystem || Report/Notes ||
| Docker | Client=17.05.0-ce Server=17.05.0-ce Image:yetus/hbase:eee3b01 |
| JIRA Issue | HBASE-19200 |
| JIRA Patch URL |

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-10 Thread Duo Zhang (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247446#comment-16247446
 ] 

Duo Zhang commented on HBASE-19200:
---

Good. We finally have the failed UTs in the report. Let me check.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200-v4.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Hadoop QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16247136#comment-16247136
 ] 

Hadoop QA commented on HBASE-19200:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
13s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Findbugs executables are not available. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 22 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
33s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
14s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
12s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  2m 
36s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  7m 
38s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
44s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
12s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  4m 
50s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  1m 
57s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  1m 
57s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
30s{color} | {color:red} hbase-client: The patch generated 4 new + 340 
unchanged - 29 fixed = 344 total (was 369) {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  1m  
6s{color} | {color:red} hbase-server: The patch generated 1 new + 575 unchanged 
- 1 fixed = 576 total (was 576) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  4m 
29s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
54m 56s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 2.7.4 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
37s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  2m 
49s{color} | {color:green} hbase-client in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
14s{color} | {color:green} hbase-replication in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}129m 19s{color} 
| {color:red} hbase-server in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
37s{color} | {color:green} hbase-rsgroup in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
45s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  1m 
31s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}220m 38s{color} | 
{color:black} {color} |
\\
\\
|| Reason || Tests ||
| Failed junit tests | 
hadoop.hbase.replication.regionserver.TestTableBasedReplicationSourceManagerImpl
 |
|   |

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Duo Zhang (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246896#comment-16246896
 ] 

Duo Zhang commented on HBASE-19200:
---

Yes, I prefer that we do not remove other zk related classes other than 
ZooKeeperRegistry, just purge the usage of them in hbase-client. We can do the 
purging in HBASE-19114. Let me address the problems.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246884#comment-16246884
 ] 

Appy commented on HBASE-19200:
--

In last QA, regionserver.TestHRegionWithInMemoryFlush is probably the culprit 
test. Checkstyles too.
I'll use your patch-in-process and re-start on my zookeeper module work.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Appy (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246882#comment-16246882
 ] 

Appy commented on HBASE-19200:
--

Thanks Andrew.
I think it's fine to leave ZKLeaderManager be. hbase-client doesn't want it, so 
can be moved to new module.
Feels like patch is already too big and close to commit. Getting it in first 
and iterating will be faster and easier.

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread stack (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246593#comment-16246593
 ] 

stack commented on HBASE-19200:
---

Thanks [~apurtell]

Doesn't look like we can purge ZKLeaderManager [~Apache9] ... 

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Andrew Purtell (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246576#comment-16246576
 ] 

Andrew Purtell commented on HBASE-19200:


[~stack] AuthenticationTokenSecretManager is used to generate the secret keys 
for HBase token based auth, supporting secure RPC for MapReduce and Spark jobs. 
This class is part of the TokenProvider coprocessor. 

> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread stack (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16246420#comment-16246420
 ] 

stack commented on HBASE-19200:
---

bq. MasterAddressTracker and MetaTableAccessor are server side only after this 
change.

Great.

ZkAclReset is used by the hbase-cleanup.sh shell script to clean all vestige of 
hbase from zk, hdfs, and from acl. We can more the tool into hbase-server?

On ZKLeaderManager, the deprecation and comment that it was not used was added 
by this:

tree a351456d4cdf90badd64dc7f4e61476595f6b74d
parent 5edfc1996d30d12d8b215ddf16ca6e39a7fc1c7d
author Michael Stack  Wed May 9 17:08:33 2012 +
committer Michael Stack  Wed May 9 17:08:33 2012 +

HBASE-5944 Convert rs, shutdown, and table dir content to pb

So, it has been deprecated for a long time. Safe to remove. 

But it is used in AuthenticationTokenSecretManager which looks like it is still 
used. [~apurtell] You know how AuthenticationTokenSecretManager is used sir? 
Thanks.



> make hbase-client only depend on ZKAsyncRegistry and ZNodePaths
> ---
>
> Key: HBASE-19200
> URL: https://issues.apache.org/jira/browse/HBASE-19200
> Project: HBase
>  Issue Type: Task
>  Components: Client, Zookeeper
>Reporter: Duo Zhang
>Assignee: Duo Zhang
> Fix For: 2.0.0-beta-1
>
> Attachments: HBASE-19200-v1.patch, HBASE-19200-v2.patch, 
> HBASE-19200-v3.patch, HBASE-19200.patch
>
>
> So that we can move most of the zookeeper related code out of hbase-client 
> module.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

[jira] [Commented] (HBASE-19200) make hbase-client only depend on ZKAsyncRegistry and ZNodePaths

2017-11-09 Thread Hadoop QA (JIRA) 

[ 
https://issues.apache.org/jira/browse/HBASE-19200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16245786#comment-16245786
 ] 

Hadoop QA commented on HBASE-19200:
---

| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m 
10s{color} | {color:blue} Docker mode activated. {color} |
|| || || || {color:brown} Prechecks {color} ||
| {color:blue}0{color} | {color:blue} findbugs {color} | {color:blue}  0m  
0s{color} | {color:blue} Findbugs executables are not available. {color} |
| {color:green}+1{color} | {color:green} hbaseanti {color} | {color:green}  0m  
0s{color} | {color:green} Patch does not have any anti-patterns. {color} |
| {color:green}+1{color} | {color:green} @author {color} | {color:green}  0m  
0s{color} | {color:green} The patch does not contain any @author tags. {color} |
| {color:green}+1{color} | {color:green} test4tests {color} | {color:green}  0m 
 0s{color} | {color:green} The patch appears to include 22 new or modified test 
files. {color} |
|| || || || {color:brown} master Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
16s{color} | {color:blue} Maven dependency ordering for branch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
40s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
39s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} checkstyle {color} | {color:green}  2m 
57s{color} | {color:green} master passed {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  8m 
11s{color} | {color:green} branch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
56s{color} | {color:green} master passed {color} |
|| || || || {color:brown} Patch Compile Tests {color} ||
| {color:blue}0{color} | {color:blue} mvndep {color} | {color:blue}  0m 
15s{color} | {color:blue} Maven dependency ordering for patch {color} |
| {color:green}+1{color} | {color:green} mvninstall {color} | {color:green}  5m 
44s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} compile {color} | {color:green}  2m 
43s{color} | {color:green} the patch passed {color} |
| {color:green}+1{color} | {color:green} javac {color} | {color:green}  2m 
43s{color} | {color:green} the patch passed {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
39s{color} | {color:red} hbase-client: The patch generated 16 new + 333 
unchanged - 29 fixed = 349 total (was 362) {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  0m 
14s{color} | {color:red} hbase-replication: The patch generated 1 new + 15 
unchanged - 0 fixed = 16 total (was 15) {color} |
| {color:red}-1{color} | {color:red} checkstyle {color} | {color:red}  1m 
20s{color} | {color:red} hbase-server: The patch generated 8 new + 575 
unchanged - 1 fixed = 583 total (was 576) {color} |
| {color:green}+1{color} | {color:green} whitespace {color} | {color:green}  0m 
 0s{color} | {color:green} The patch has no whitespace issues. {color} |
| {color:green}+1{color} | {color:green} shadedjars {color} | {color:green}  5m 
12s{color} | {color:green} patch has no errors when building our shaded 
downstream artifacts. {color} |
| {color:green}+1{color} | {color:green} hadoopcheck {color} | {color:green} 
61m 48s{color} | {color:green} Patch does not cause any errors with Hadoop 
2.6.1 2.6.2 2.6.3 2.6.4 2.6.5 2.7.1 2.7.2 2.7.3 2.7.4 or 3.0.0-alpha4. {color} |
| {color:green}+1{color} | {color:green} javadoc {color} | {color:green}  1m 
56s{color} | {color:green} the patch passed {color} |
|| || || || {color:brown} Other Tests {color} ||
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  3m 
21s{color} | {color:green} hbase-client in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
16s{color} | {color:green} hbase-replication in the patch passed. {color} |
| {color:red}-1{color} | {color:red} unit {color} | {color:red}159m 36s{color} 
| {color:red} hbase-server in the patch failed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  1m 
51s{color} | {color:green} hbase-rsgroup in the patch passed. {color} |
| {color:green}+1{color} | {color:green} unit {color} | {color:green}  0m 
48s{color} | {color:green} hbase-it in the patch passed. {color} |
| {color:green}+1{color} | {color:green} asflicense {color} | {color:green}  1m 
46s{color} | {color:green} The patch does not generate ASF License warnings. 
{color} |
| {color:black}{color} | {color:black} {color} | {color:black}262m 46s{color} |