Whirr
  1. Whirr
  2. WHIRR-608

CDH HBase configuration uses CDH3 package and service names only

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.7.1
    • Fix Version/s: 0.8.0
    • Component/s: service/cdh
    • Labels:
      None

      Description

      The service and package names in configure_cdh_hbase.sh and install_cdh_hbase.sh use the CDH3-style naming - i.e., "hadoop-hbase", "hadoop-hbase-master", etc. They should be changed to switch between the CDH3 namees and CDH4 names depending on whether we're using CDH3 or CDH4.

      1. WHIRR-608.patch
        3 kB
        Andrew Bayer
      2. WHIRR-608-2.patch
        3 kB
        Bruno Dumon

        Activity

        Hide
        Tom White added a comment -

        +1

        Show
        Tom White added a comment - +1
        Hide
        Bruno Dumon added a comment -

        same as previous patch but with cdh4 as default REPO in install_cdh_hbase.sh

        Show
        Bruno Dumon added a comment - same as previous patch but with cdh4 as default REPO in install_cdh_hbase.sh
        Hide
        Bruno Dumon added a comment -

        I happened to make almost exactly the same patch yesterday

        I've attached your patch with one modification: cdh4 as default repo in install_cdh_hbase.sh, as this is also the default in the other scripts. (not that it matters much, since REPO is a global var).

        Show
        Bruno Dumon added a comment - I happened to make almost exactly the same patch yesterday I've attached your patch with one modification: cdh4 as default repo in install_cdh_hbase.sh, as this is also the default in the other scripts. (not that it matters much, since REPO is a global var).
        Hide
        Andrew Bayer added a comment -

        +1 on that - I'll commit it tomorrow, as for some reason Cloudera's offices are blacklisted from ASF SVN at the moment. =)

        Show
        Andrew Bayer added a comment - +1 on that - I'll commit it tomorrow, as for some reason Cloudera's offices are blacklisted from ASF SVN at the moment. =)
        Hide
        Adrian Cole added a comment -

        Committed revision 1369249.

        Show
        Adrian Cole added a comment - Committed revision 1369249.

          People

          • Assignee:
            Andrew Bayer
            Reporter:
            Andrew Bayer
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development