You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Michael Stack (Jira)" <ji...@apache.org> on 2021/03/17 17:55:00 UTC

[jira] [Updated] (HBASE-25676) Move generic classes from hbase-server to hbase-common

     [ https://issues.apache.org/jira/browse/HBASE-25676?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Michael Stack updated HBASE-25676:
----------------------------------
    Description: 
There's a bunch of classes that are not hbase-server specific on cursory review that could live in hbase-common... not many, about 3% of src/main/java but move them out.

{code}
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/SslRMIClientSocketFactorySecure.java (99%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/SslRMIServerSocketFactorySecure.java (99%)
  rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthCheckChore.java (93%)
  rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthChecker.java (86%)
  rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthReport.java (94%)
  rename {hbase-server/src/test/java/org/apache/hadoop/hbase => hbase-common/src/test/java/org/apache/hadoop/hbase/healthcheck}/TestNodeHealthCheckChore.java (86%)
  delete mode 100644 hbase-server/src/main/java/org/apache/hadoop/hbase/DaemonThreadFactory.java
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/security/SecurityUtil.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ConfigurationUtil.java (99%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/DirectMemoryUtils.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/GetJavaProperty.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/HBaseConfTool.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/HashedBytes.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/IdReadWriteLock.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/JvmVersion.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/KeyRange.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/LossyCounting.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ManualEnvironmentEdge.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/MunkresAssignment.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/NettyEventLoopGroupConfig.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/RegionSplitCalculator.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/RollingStatCalculator.java (99%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ShutdownHookManager.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/SortedList.java (100%)
  rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/StealJobQueue.java (100%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestConfigurationUtil.java (100%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestIdReadWriteLock.java (100%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestLossyCounting.java (100%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestRegionSplitCalculator.java (95%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestSortedList.java (100%)
  rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestStealJobQueue.java (100%)
{code}

  was:There's a bunch of classes that are not hbase-server specific on cursory review that could live in hbase-common... not many, about 3% of src/main/java but move them out.


> Move generic classes from hbase-server to hbase-common
> ------------------------------------------------------
>
>                 Key: HBASE-25676
>                 URL: https://issues.apache.org/jira/browse/HBASE-25676
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Michael Stack
>            Priority: Major
>
> There's a bunch of classes that are not hbase-server specific on cursory review that could live in hbase-common... not many, about 3% of src/main/java but move them out.
> {code}
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/SslRMIClientSocketFactorySecure.java (99%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/SslRMIServerSocketFactorySecure.java (99%)
>   rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthCheckChore.java (93%)
>   rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthChecker.java (86%)
>   rename {hbase-server/src/main/java/org/apache/hadoop/hbase => hbase-common/src/main/java/org/apache/hadoop/hbase/healthcheck}/HealthReport.java (94%)
>   rename {hbase-server/src/test/java/org/apache/hadoop/hbase => hbase-common/src/test/java/org/apache/hadoop/hbase/healthcheck}/TestNodeHealthCheckChore.java (86%)
>   delete mode 100644 hbase-server/src/main/java/org/apache/hadoop/hbase/DaemonThreadFactory.java
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/security/SecurityUtil.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ConfigurationUtil.java (99%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/DirectMemoryUtils.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/GetJavaProperty.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/HBaseConfTool.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/HashedBytes.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/IdReadWriteLock.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/JvmVersion.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/KeyRange.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/LossyCounting.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ManualEnvironmentEdge.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/MunkresAssignment.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/NettyEventLoopGroupConfig.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/RegionSplitCalculator.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/RollingStatCalculator.java (99%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/ShutdownHookManager.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/SortedList.java (100%)
>   rename {hbase-server => hbase-common}/src/main/java/org/apache/hadoop/hbase/util/StealJobQueue.java (100%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestConfigurationUtil.java (100%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestIdReadWriteLock.java (100%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestLossyCounting.java (100%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestRegionSplitCalculator.java (95%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestSortedList.java (100%)
>   rename {hbase-server => hbase-common}/src/test/java/org/apache/hadoop/hbase/util/TestStealJobQueue.java (100%)
> {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)