hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hadoop QA (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-19114) Split out o.a.h.h.zookeeper from hbase-server and hbase-client
Date Tue, 31 Oct 2017 19:53:00 GMT

    [ https://issues.apache.org/jira/browse/HBASE-19114?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16227403#comment-16227403

Hadoop QA commented on HBASE-19114:

| (x) *{color:red}-1 overall{color}* |
|| Vote || Subsystem || Runtime || Comment ||
| {color:blue}0{color} | {color:blue} reexec {color} | {color:blue}  0m  0s{color} | {color:blue}
Docker mode activated. {color} |
| {color:red}-1{color} | {color:red} patch {color} | {color:red}  0m  7s{color} | {color:red}
HBASE-19114 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/0.5.0/precommit-patchnames
for help. {color} |
|| Subsystem || Report/Notes ||
| JIRA Issue | HBASE-19114 |
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12895051/HBASE-19114.master.004.patch
| Console output | https://builds.apache.org/job/PreCommit-HBASE-Build/9540/console |
| Powered by | Apache Yetus 0.5.0   http://yetus.apache.org |

This message was automatically generated.

> Split out o.a.h.h.zookeeper from hbase-server and hbase-client
> --------------------------------------------------------------
>                 Key: HBASE-19114
>                 URL: https://issues.apache.org/jira/browse/HBASE-19114
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Appy
>            Assignee: Appy
>         Attachments: HBASE-19114.master.001.patch, HBASE-19114.master.002.patch, HBASE-19114.master.003.patch,
> Changes so far:
> - Moved DrainingServerTracker and RegionServerTracker to hbase-server:o.a.h.h.master.
> - Move Abortable to hbase-common. Since it's IA.Private and independent of anything,
moving it to hbase-common which is at bottom of the dependency tree is better.
> - Moved RecoveringRegionWatcher to hbase-server:o.a.h.h.regionserver
> - Moved SplitOrMergeTracker to oahh.master (because it depends on a PB)
> - Moving hbase-client:oahh.zookeeper.*  to hbase-zookeeper module. We want to keep hbase-zookeeper
very independent and hence at lowest levels in our dependency tree.
> - ZKUtil is a huge tangle since it's linked to almost everything in \[hbase-client/]oahh.zookeeper.
And pulling it down requires some basic proto functions (mergeFrom, PBmagic, etc). So what
i did was:
>    ** Pulled down common and basic protobuf functions (which only depend on com.google.protobuf.\*)
to hbase-common so other code depending on them can be pulled down if possible/wanted in future.
This will help future dependency untangling too. These are ProtobufMagic and ProtobufHelpers.
>   ** Didn't move any hbase-specific PB stuff to hbase-common. We can't pull things into
hbase-common which add dependency between it and hbase-protobuf/hbase-shaded-protobuf since
we very recently untangled them.
> - DEFAULT_REPLICA_ID is used in many places in ZK. Declared a new constant in HConstants
(since it's in hbase-common) and using it in hbase-zookeeper. RegionInfo.DEFAULT_REPLICA_ID
also takes its value from it to avoid case where two values can become different.
> - Renamed some classes to use a consistent naming for classes - ZK instead of mix of
ZK, Zk , ZooKeeper. Couldn't rename following public classes: MiniZooKeeperCluster, ZooKeeperConnectionException.

This message was sent by Atlassian JIRA

View raw message