Uploaded image for project: 'Bigtop'
  1. Bigtop
  2. BIGTOP-1382

Hadoop site.csv components additions: yarn & mapred-app

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: backlog, 0.7.0
    • Fix Version/s: 0.8.0
    • Component/s: vm
    • Labels:

      Description

      Added yarn and mapred-app to site.csv component list in /vagrant-puppet/provision.sh

      1. BIGTOP-1382.patch
        0.8 kB
        Jon Cope
      2. BIGTOP-1382.patch
        1 kB
        Jon Cope

        Activity

        Hide
        jayunit100 jay vyas added a comment -

        Hi jon. Looks like you got some extra stuff in the patch still.

        You'll have to redo the patch. The simplest way:

        • clone bigtop from scratch or create a branch from master
        • reintroduce your change
        • run the git commit / format-patch command as you did before.

        Thanks

        Show
        jayunit100 jay vyas added a comment - Hi jon. Looks like you got some extra stuff in the patch still. You'll have to redo the patch. The simplest way: clone bigtop from scratch or create a branch from master reintroduce your change run the git commit / format-patch command as you did before. Thanks
        Hide
        jayunit100 jay vyas added a comment -

        Hi jon ! You DEFINETLY do NOT want to put in that change to mapreduce_shuffle.

        The reason is that for 0.8.0 (head) release, the parameter is correct.
        You are deploying 0.7.0, where you manually change that parameter, but we dont want to bake that change into the puppet recipes.

        Makes sense? otherwise, I like adding yarn and mapred components.

        Please revert the change to the xml file, and maybe replace it with a comment specifying that for 0.7.0, this should be mapreduce.shuffle, for 0.8+, mapreduce_shuffle .

        Show
        jayunit100 jay vyas added a comment - Hi jon ! You DEFINETLY do NOT want to put in that change to mapreduce_shuffle. The reason is that for 0.8.0 (head) release, the parameter is correct. You are deploying 0.7.0 , where you manually change that parameter, but we dont want to bake that change into the puppet recipes. Makes sense? otherwise, I like adding yarn and mapred components. Please revert the change to the xml file, and maybe replace it with a comment specifying that for 0.7.0, this should be mapreduce.shuffle, for 0.8+, mapreduce_shuffle .
        Hide
        jcope Jon Cope added a comment -

        Ah, okay! Reverting the change and recreating.

        Show
        jcope Jon Cope added a comment - Ah, okay! Reverting the change and recreating.
        Hide
        jayunit100 jay vyas added a comment -

        +1 thanks this is a minor but important update that will allow people to easily try out bigtop using vagrant.

        ill commit this shortly

        Show
        jayunit100 jay vyas added a comment - +1 thanks this is a minor but important update that will allow people to easily try out bigtop using vagrant. ill commit this shortly
        Hide
        jayunit100 jay vyas added a comment -

        and commited.

        Show
        jayunit100 jay vyas added a comment - and commited.

          People

          • Assignee:
            jcope Jon Cope
            Reporter:
            jcope Jon Cope
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development