[ https://issues.apache.org/jira/browse/HBASE-6226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13403574#comment-13403574 ]
Mikhail Bautin commented on HBASE-6226: --------------------------------------- Matt: regarding the Arcanist issue: have you run "mvn -Darc initialize"? It would download and install .arc_jira_lib in your HBase directory. Phabricator is a more convenient code review system compared to ReviewBoard or reviewing raw patches, because e.g. it correctly identifies renamed files (relevant to this particular diff). > move DataBlockEncoding and related classes to hbase-common module > ----------------------------------------------------------------- > > Key: HBASE-6226 > URL: https://issues.apache.org/jira/browse/HBASE-6226 > Project: HBase > Issue Type: Improvement > Components: io, regionserver > Affects Versions: 0.96.0 > Reporter: Matt Corgan > Assignee: Matt Corgan > Fix For: 0.96.0 > > Attachments: HBASE-6226-v1.patch, HBASE-6226-v2.patch, > HBASE-6226-v3.patch, HBASE-6226-v4.patch > > > In order to isolate the implementation details of HBASE-4676 (PrefixTrie > encoding) and other DataBlockEncoders by putting them in modules, this pulls > up the DataBlockEncoding related interfaces into hbase-common. > No tests are moved in this patch. The only notable change was trimming a few > dependencies on HFileBlock which adds dependencies to much of the > regionserver. > The test suite passes locally for me. > I tried to keep it as simple as possible... let me know if there are any > concerns. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira