Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-4336

Convert source tree into maven modules



    • Task
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • None
    • 0.95.0
    • build
    • None


      When we originally converted the build to maven we had a single "core" module defined, but later reverted this to a module-less build for the sake of simplicity.

      It now looks like it's time to re-address this, as we have an actual need for modules to:

      • provide a trimmed down "client" library that applications can make use of
      • more cleanly support building against different versions of Hadoop, in place of some of the reflection machinations currently required
      • incorporate the secure RPC engine that depends on some secure Hadoop classes

      I propose we start simply by refactoring into two initial modules:

      • core - common classes and utilities, and client-side code and interfaces
      • server - master and region server implementations and supporting code

      This would also lay the groundwork for incorporating the HBase security features that have been developed. Once the module structure is in place, security-related features could then be incorporated into a third module – "security" – after normal review and approval. The security module could then depend on secure Hadoop, without modifying the dependencies of the rest of the HBase code.


        1. 4336-addendum-2.txt
          334 kB
          Ted Yu
        2. 4336-addendum-3.txt
          334 kB
          Ted Yu
        3. 4336v2_part2.txt
          131 kB
          Michael Stack
        4. 4336v2.txt.gz
          5.24 MB
          Michael Stack
        5. hbase-4336-addendum.patch
          5 kB
          Jesse Yates
        6. hbase-4336-v1.patch.gz
          5.24 MB
          Jesse Yates
        7. hbase-4336-v2.patch.gz
          5.24 MB
          Jesse Yates
        8. refactor.txt.gz
          4.90 MB
          Michael Stack

        Issue Links



              jesse_yates Jesse Yates
              ghelmling Gary Helmling
              0 Vote for this issue
              15 Start watching this issue