Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.11
    • Fix Version/s: 0.12.0
    • Component/s: None
    • Labels:
      None

      Description

      Currently, PigUnit is not thread-safe because Cluster and PigServer are declared as static. Converting them to ThreadLocal allows PigUnit to run in multi-threaded environment.

      1. PIG-3096.patch
        4 kB
        Cheolsoo Park

        Issue Links

          Activity

          Cheolsoo Park created issue -
          Hide
          Cheolsoo Park added a comment -

          Attached is a patch that does the following:

          • Convert Cluster and PigServer to ThreadLocal
          • Randomize the destination of input file in the cluster so that input files don't conflict when multiple jobs run in parallel.
          Show
          Cheolsoo Park added a comment - Attached is a patch that does the following: Convert Cluster and PigServer to ThreadLocal Randomize the destination of input file in the cluster so that input files don't conflict when multiple jobs run in parallel.
          Cheolsoo Park made changes -
          Field Original Value New Value
          Attachment PIG-3096.patch [ 12561232 ]
          Cheolsoo Park made changes -
          Remote Link This issue links to "Review Board (Web Link)" [ 11910 ]
          Cheolsoo Park made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Bill Graham added a comment -

          +1 lgtm.

          Show
          Bill Graham added a comment - +1 lgtm.
          Hide
          Cheolsoo Park added a comment -

          Thanks Bill for reviewing it.

          I committed it to trunk.

          Show
          Cheolsoo Park added a comment - Thanks Bill for reviewing it. I committed it to trunk.
          Cheolsoo Park made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Daniel Dai made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          10m 34s 1 Cheolsoo Park 17/Dec/12 01:35
          Patch Available Patch Available Resolved Resolved
          3d 14h 56m 1 Cheolsoo Park 20/Dec/12 16:31
          Resolved Resolved Closed Closed
          298d 14m 1 Daniel Dai 14/Oct/13 16:46

            People

            • Assignee:
              Cheolsoo Park
              Reporter:
              Cheolsoo Park
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development