Bigtop
  1. Bigtop
  2. BIGTOP-351

Make HBASE_HEAPSIZE manageable via puppet

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.4.0
    • Fix Version/s: 0.5.0
    • Component/s: deployment
    • Labels:
      None

      Description

      Previously, HBASE_HEAPSIZE was not being set in /etc/hbase/conf/hbase-env.sh by puppet. I'm attaching a patch which adds this, such that it's configurable from site.csv.

      1. patch.txt
        3 kB
        Patrick Taylor Ramsey

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        215d 21h 45m 1 Roman Shaposhnik 20/Aug/12 19:24
        Resolved Resolved Closed Closed
        305d 5h 30m 1 Roman Shaposhnik 22/Jun/13 00:55
        Roman Shaposhnik made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Roman Shaposhnik made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Patrick Taylor Ramsey [ ptr ]
        Resolution Fixed [ 1 ]
        Roman Shaposhnik made changes -
        Fix Version/s 0.5.0 [ 12321865 ]
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12649410 ] patch-available, re-open possible [ 12665738 ]
        Hide
        Roman Shaposhnik added a comment -

        Thanks for the patch! Committed.

        Show
        Roman Shaposhnik added a comment - Thanks for the patch! Committed.
        Hide
        Bruno Mahé added a comment -

        +1
        Isn't there a way to reference the same constant from all these places?

        Show
        Bruno Mahé added a comment - +1 Isn't there a way to reference the same constant from all these places?
        Patrick Taylor Ramsey made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Patrick Taylor Ramsey made changes -
        Field Original Value New Value
        Attachment patch.txt [ 12510899 ]
        Patrick Taylor Ramsey created issue -

          People

          • Assignee:
            Patrick Taylor Ramsey
            Reporter:
            Patrick Taylor Ramsey
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development