Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.6.0
    • Component/s: None
    • Labels:
      None

      Description

      HBase currently has 3 platform-dependent init script templates. Migrating to use the same template that Sqoop now uses will simplify maintenance now, and ease the transition to SysV init replacements in the future.

        Activity

        Hide
        Roman Shaposhnik added a comment -

        That'll be pretty awesome to have for 0.6.0

        Show
        Roman Shaposhnik added a comment - That'll be pretty awesome to have for 0.6.0
        Hide
        Sean Mackrory added a comment -

        That very much depends on the timing of 0.6.0, but I'll do my best to get this done within the next couple of days.

        Show
        Sean Mackrory added a comment - That very much depends on the timing of 0.6.0, but I'll do my best to get this done within the next couple of days.
        Hide
        Sean Mackrory added a comment -

        I've tested this patch successfully on Ubuntu 12, SLES 11 and RHEL 6. It's intended to be applied over BIGTOP-732, after which the regionserver will still use its own template (although it was originally distro-agnostic anyway).

        Show
        Sean Mackrory added a comment - I've tested this patch successfully on Ubuntu 12, SLES 11 and RHEL 6. It's intended to be applied over BIGTOP-732 , after which the regionserver will still use its own template (although it was originally distro-agnostic anyway).
        Hide
        Sean Mackrory added a comment -

        As this is a larger patch, I've uploaded it to ReviewBoard if anyone wants to use that: https://reviews.apache.org/r/10778/

        Show
        Sean Mackrory added a comment - As this is a larger patch, I've uploaded it to ReviewBoard if anyone wants to use that: https://reviews.apache.org/r/10778/
        Hide
        Sean Mackrory added a comment -

        My apologies - some changes appear to have been lost between me testing and submitting the patch. I'll redo my testing to make sure nothing else is lost, but I think this is it..

        Show
        Sean Mackrory added a comment - My apologies - some changes appear to have been lost between me testing and submitting the patch. I'll redo my testing to make sure nothing else is lost, but I think this is it..
        Hide
        Mark Grover added a comment -

        Left some minor comments on RB but +1 (non-committer)

        Show
        Mark Grover added a comment - Left some minor comments on RB but +1 (non-committer)
        Hide
        Mark Grover added a comment -

        +1 (non-committer)

        Show
        Mark Grover added a comment - +1 (non-committer)
        Hide
        Roman Shaposhnik added a comment -

        +1 and committed! NOTE: I did change CONF_DIR to be "/etc/hbase/conf" – I understand the need for flexibility but lets have it as a separate patch.

        Show
        Roman Shaposhnik added a comment - +1 and committed! NOTE: I did change CONF_DIR to be "/etc/hbase/conf" – I understand the need for flexibility but lets have it as a separate patch.

          People

          • Assignee:
            Sean Mackrory
            Reporter:
            Sean Mackrory
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development