Pig
  1. Pig
  2. PIG-2702

Make Pig local mode (and tests) faster by working around the hard coded sleep(5000) in hadoop's JobControl

    Details

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

      Description

      the JobControl class polls the jobs statuses with a hard coded sleep(5000) making every job take a minimum of 5 seconds.
      Since the local mode uses the same code path this makes local mode (and unit tests) extremely slow.

      1. PIG-2702_a.patch
        23 kB
        Julien Le Dem
      2. PIG-2702.patch
        20 kB
        Julien Le Dem

        Activity

        Hide
        Dmitriy V. Ryaboy added a comment -

        Ahh finally .
        Don't forget that we need to add apache headers to new files though.

        Show
        Dmitriy V. Ryaboy added a comment - Ahh finally . Don't forget that we need to add apache headers to new files though.
        Hide
        Julien Le Dem added a comment -

        PIG-2702_a.patch
        Adding apache headers

        Show
        Julien Le Dem added a comment - PIG-2702 _a.patch Adding apache headers
        Hide
        Dmitriy V. Ryaboy added a comment -

        Without any modifications, speed of test-commit before patch:

        BUILD SUCCESSFUL
        Total time: 22 minutes 38 seconds
        
        real	22m38.364s
        user	10m16.580s
        sys	1m34.482s
        

        And after patch, it goes 18% faster.

        BUILD SUCCESSFUL
        Total time: 18 minutes 36 seconds
        
        real	18m36.409s
        user	10m15.993s
        sys	1m36.040s
        

        Moving more tests to use local mode will give us further speed improvements.

        Show
        Dmitriy V. Ryaboy added a comment - Without any modifications, speed of test-commit before patch: BUILD SUCCESSFUL Total time: 22 minutes 38 seconds real 22m38.364s user 10m16.580s sys 1m34.482s And after patch, it goes 18% faster. BUILD SUCCESSFUL Total time: 18 minutes 36 seconds real 18m36.409s user 10m15.993s sys 1m36.040s Moving more tests to use local mode will give us further speed improvements.
        Hide
        Dmitriy V. Ryaboy added a comment -

        +1

        Show
        Dmitriy V. Ryaboy added a comment - +1

          People

          • Assignee:
            Julien Le Dem
            Reporter:
            Julien Le Dem
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development