Pig
  1. Pig
  2. PIG-124

only run one test (aant runtest -Dtest=TestMapReduce) not the complete test suite

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.1.0
    • Component/s: None
    • Labels:
      None
    • Patch Info:
      Patch Available

      Description

      +1 to what Xu is saying.

      1. RunIndividualTestCase.patch
        2 kB
        Xu Zhang
      2. PIG-124_v_1.patch
        4 kB
        Stefan Groschupf

        Activity

        Stefan Groschupf created issue -
        Hide
        Stefan Groschupf added a comment -

        another build.xml improvement again r631029.

        Show
        Stefan Groschupf added a comment - another build.xml improvement again r631029.
        Stefan Groschupf made changes -
        Field Original Value New Value
        Attachment PIG-124_v_1.patch [ 12376472 ]
        Hide
        Pi Song added a comment -

        +1 I've been looking forward to this.

        Show
        Pi Song added a comment - +1 I've been looking forward to this.
        Hide
        Xu Zhang added a comment - - edited

        As described in the last bullet of section "Making Changes" in the HowToContribute doc ( http://wiki.apache.org/pig/HowToContribute ), Pig does (or is supposed to ) support the following command for running an individual testcase class,

        ant test -Dtestcase=<ClassName>  (For example ant test -Dtestcase=TestPigFile)
        

        although it was not working last I checked. Personally, I would prefer it this way since this command conforms to what Hadoop uses to achieve the same purpose. To make work, what we need to do is just port the same functionality from Hadoop's build.xml to Pig's build.xml.

        So if there is no objections, either Stefan (since Stefan has been doing a great job on this already) or myself can start making the above command work as expected.

        Show
        Xu Zhang added a comment - - edited As described in the last bullet of section "Making Changes" in the HowToContribute doc ( http://wiki.apache.org/pig/HowToContribute ), Pig does (or is supposed to ) support the following command for running an individual testcase class, ant test -Dtestcase=<ClassName> (For example ant test -Dtestcase=TestPigFile) although it was not working last I checked. Personally, I would prefer it this way since this command conforms to what Hadoop uses to achieve the same purpose. To make work, what we need to do is just port the same functionality from Hadoop's build.xml to Pig's build.xml. So if there is no objections, either Stefan (since Stefan has been doing a great job on this already) or myself can start making the above command work as expected.
        Olga Natkovich made changes -
        Patch Info [Patch Available]
        Description +1 to what Xu is saying.
        Hide
        Xu Zhang added a comment -

        Attach a patch that makes build.xml able to run the individual test case class.

        Show
        Xu Zhang added a comment - Attach a patch that makes build.xml able to run the individual test case class.
        Xu Zhang made changes -
        Attachment RunIndividualTestCase.patch [ 12376547 ]
        Hide
        Stefan Groschupf added a comment -

        +1
        Xu: cool - even better than my solution. Thanks!
        Lets get it in soon - waiting 12 minutes to run a test every time is painful.

        Show
        Stefan Groschupf added a comment - +1 Xu: cool - even better than my solution. Thanks! Lets get it in soon - waiting 12 minutes to run a test every time is painful.
        Hide
        Xu Zhang added a comment -

        Credit goes to whoever did the similar thing for Hadoop.

        Show
        Xu Zhang added a comment - Credit goes to whoever did the similar thing for Hadoop .
        Hide
        Arun C Murthy added a comment -

        Xu, can you also add the 'test.output' flag to pig's build.xml, similar to hadoop? This ensures that we get output on the screen while the test case is running... which is usually very useful for debugging. Thanks!

        Show
        Arun C Murthy added a comment - Xu, can you also add the 'test.output' flag to pig's build.xml, similar to hadoop? This ensures that we get output on the screen while the test case is running... which is usually very useful for debugging. Thanks!
        Hide
        Xu Zhang added a comment -

        Sure. I am working on it now.

        Show
        Xu Zhang added a comment - Sure. I am working on it now.
        Xu Zhang made changes -
        Attachment RunIndividualTestCase.patch [ 12376547 ]
        Hide
        Xu Zhang added a comment -

        Here you go, Arun. To use it, on the command line specify the option "-Dtest.ouput=yes". I will update the related documents for this option accordingly.

        Show
        Xu Zhang added a comment - Here you go, Arun. To use it, on the command line specify the option "-Dtest.ouput=yes". I will update the related documents for this option accordingly.
        Xu Zhang made changes -
        Attachment RunIndividualTestCase.patch [ 12376581 ]
        Olga Natkovich made changes -
        Patch Info [Patch Available]
        Hide
        Olga Natkovich added a comment -

        Patch committed. Xu, thanks for contributing

        Show
        Olga Natkovich added a comment - Patch committed. Xu, thanks for contributing
        Olga Natkovich made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Owen O'Malley made changes -
        Workflow jira [ 12424468 ] no-reopen-closed, patch-avail [ 12425431 ]
        Olga Natkovich made changes -
        Fix Version/s 0.1.0 [ 12312848 ]
        Alan Gates made changes -
        Assignee Stefan Groschupf [ joa23 ]
        Alan Gates made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        1d 15h 9m 1 Olga Natkovich 28/Feb/08 01:04
        Resolved Resolved Closed Closed
        755d 20h 56m 1 Alan Gates 24/Mar/10 22:01

          People

          • Assignee:
            Stefan Groschupf
            Reporter:
            Stefan Groschupf
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development