Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.7.0
    • Fix Version/s: 0.7.1
    • Component/s: None
    • Labels:
    • Environment:

      Linux

      Description

      I think it would be better if whirr made hadoop-proxy.sh executable. This would ease its invocation. IMHO

        Activity

        Hide
        Andrei Savu added a comment -

        Thanks Marco for reporting. BTW it's easy to contribute:
        https://cwiki.apache.org/confluence/display/WHIRR/How+To+Contribute

        Show
        Andrei Savu added a comment - Thanks Marco for reporting. BTW it's easy to contribute: https://cwiki.apache.org/confluence/display/WHIRR/How+To+Contribute
        Hide
        Marco Didonna added a comment -

        next time I'll post a patch, I swear to god

        Show
        Marco Didonna added a comment - next time I'll post a patch, I swear to god
        Hide
        Marco Didonna added a comment -

        This should fix the problem

        Show
        Marco Didonna added a comment - This should fix the problem
        Hide
        Marco Didonna added a comment - - edited

        Patch for WHIRR-439

        Show
        Marco Didonna added a comment - - edited Patch for WHIRR-439
        Hide
        Andrei Savu added a comment -

        Looks good. Can you make sure we do the same for HBase [1]?

        [1] services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java

        ps: generate the patch by running either "svn diff" or "git diff --no-prefix trunk" from project root. The one you've submitted has an absolute path.

        Show
        Andrei Savu added a comment - Looks good. Can you make sure we do the same for HBase [1] ? [1] services/hbase/src/main/java/org/apache/whirr/service/hbase/HBaseMasterClusterActionHandler.java ps: generate the patch by running either "svn diff" or "git diff --no-prefix trunk" from project root. The one you've submitted has an absolute path.
        Hide
        Marco Didonna added a comment -

        Patch file for both hadoop service and hbase service

        Show
        Marco Didonna added a comment - Patch file for both hadoop service and hbase service
        Hide
        Andrei Savu added a comment -

        Committed to trunk. Thanks Marco! (and welcome to the team )

        Show
        Andrei Savu added a comment - Committed to trunk. Thanks Marco! (and welcome to the team )
        Hide
        Andrei Savu added a comment -

        Committed to 0.7 branch.

        Show
        Andrei Savu added a comment - Committed to 0.7 branch.

          People

          • Assignee:
            Marco Didonna
            Reporter:
            Marco Didonna
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development