Bigtop
  1. Bigtop
  2. BIGTOP-936

Use init script templates for hbase

    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

        Sean Mackrory created issue -
        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.
        Sean Mackrory made changes -
        Field Original Value New Value
        Attachment 0001-BIGTOP-936.-Use-init-script-templates-for-HBase.patch [ 12580544 ]
        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).
        Sean Mackrory made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        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/
        Sean Mackrory made changes -
        Attachment 0001-BIGTOP-936.-Use-init-script-templates-for-HBase.patch [ 12580544 ]
        Sean Mackrory made changes -
        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..
        Sean Mackrory made changes -
        Sean Mackrory made changes -
        Fix Version/s 0.6.0 [ 12323895 ]
        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)
        Sean Mackrory made changes -
        Hide
        Mark Grover added a comment -

        +1 (non-committer)

        Show
        Mark Grover added a comment - +1 (non-committer)
        Sean Mackrory made changes -
        Priority Major [ 3 ] Critical [ 2 ]
        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.
        Roman Shaposhnik made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Roman Shaposhnik made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        5d 23h 41m 1 Sean Mackrory 25/Apr/13 16:58
        Patch Available Patch Available Resolved Resolved
        6d 8h 1m 1 Roman Shaposhnik 02/May/13 00:59
        Resolved Resolved Closed Closed
        50d 23h 50m 1 Roman Shaposhnik 22/Jun/13 00:49

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development