Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-2147

JobInProgress has some redundant lines in its ctor

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 0.21.0
    • Fix Version/s: 0.22.0
    • Component/s: jobtracker
    • Labels:
      None
    • Environment:

      Not environment related.

    • Hadoop Flags:
      Reviewed
    • Release Note:
      Remove some redundant lines from JobInProgress's constructor which was re-initializing things unnecessarily.
    • Tags:
      redundant code

      Description

      In the ctor of JobInProgress class that's used by the JT, lines that create the various lists of TIPs are repeated for no purpose. Might've been due to an overlook I think.

      Attaching a patch that removes these unnecessary repeats of re-init.

        Activity

        Doug Cutting made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s 0.22.0 [ 12314184 ]
        Fix Version/s 0.21.1 [ 12315272 ]
        Resolution Fixed [ 1 ]
        Doug Cutting made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Hadoop Flags [Reviewed]
        Assignee Harsh J Chouraria [ qwertymaniac ]
        Harsh J made changes -
        Field Original Value New Value
        Attachment repeated.lines.JIP.r1.diff [ 12457819 ]
        Harsh J created issue -

          People

          • Assignee:
            Harsh J
            Reporter:
            Harsh J
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development