Pig
  1. Pig
  2. PIG-1791

System property mapred.output.compress, but pig-cluster-hadoop-site.xml doesn't

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 0.8.1
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      In PIG-1714, we allow user to set system property mapred.output.compress. It also works in pig.properties. However, the same entry in pig-cluster-hadoop-site.xml is ignored.

        Issue Links

          Activity

          Hide
          Daniel Dai added a comment -

          It's tricky to test it automatically. Manual test pass.

          Show
          Daniel Dai added a comment - It's tricky to test it automatically. Manual test pass.
          Hide
          Olga Natkovich added a comment -

          +1

          Show
          Olga Natkovich added a comment - +1
          Hide
          Daniel Dai added a comment -

          Test-patch result:
          [exec] -1 overall.
          [exec]
          [exec] +1 @author. The patch does not contain any @author tags.
          [exec]
          [exec] -1 tests included. The patch doesn't appear to include any new or modified tests.
          [exec] Please justify why no tests are needed for this patch.
          [exec]
          [exec] +1 javadoc. The javadoc tool did not generate any warning messages.
          [exec]
          [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings.
          [exec]
          [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings.
          [exec]
          [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings.

          Unit-test:
          all pass.

          Manual test:
          manually tested put mapred.output.compress in pig-cluster-hadoop-site.xml, works

          Show
          Daniel Dai added a comment - Test-patch result: [exec] -1 overall. [exec] [exec] +1 @author. The patch does not contain any @author tags. [exec] [exec] -1 tests included. The patch doesn't appear to include any new or modified tests. [exec] Please justify why no tests are needed for this patch. [exec] [exec] +1 javadoc. The javadoc tool did not generate any warning messages. [exec] [exec] +1 javac. The applied patch does not increase the total number of javac compiler warnings. [exec] [exec] +1 findbugs. The patch does not introduce any new Findbugs warnings. [exec] [exec] +1 release audit. The applied patch does not increase the total number of release audit warnings. Unit-test: all pass. Manual test: manually tested put mapred.output.compress in pig-cluster-hadoop-site.xml, works
          Hide
          Daniel Dai added a comment -

          Patch committed to both trunk and 0.8 branch.

          Show
          Daniel Dai added a comment - Patch committed to both trunk and 0.8 branch.

            People

            • Assignee:
              Daniel Dai
              Reporter:
              Daniel Dai
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development