Details

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

      Description

      Generate JobHistoryEvents for the correct transitions. Fix missing / incorrect values being set.

      1. MR2582_2.patch
        32 kB
        Siddharth Seth
      2. MR2582.patch
        29 kB
        Siddharth Seth

        Activity

        Hide
        Siddharth Seth added a comment -

        Fixes event generation based on transitions.
        *FinishedEvents used for successful jobs/tasks. Unsuccessful/Failed events used for the rest.
        Fixed several missing fields - task startTime, splitLocations, incorrect status, etc)

        Marked what is pending with TODOs -> shuffleFinishTime, etc which will be fixed along with changes to TaskAttemptStatus

        Show
        Siddharth Seth added a comment - Fixes event generation based on transitions. *FinishedEvents used for successful jobs/tasks. Unsuccessful/Failed events used for the rest. Fixed several missing fields - task startTime, splitLocations, incorrect status, etc) Marked what is pending with TODOs -> shuffleFinishTime, etc which will be fixed along with changes to TaskAttemptStatus
        Hide
        Sharad Agarwal added a comment -

        With the patch, the job link on history server UI does not work. Can you try out Siddharth ?

        Show
        Sharad Agarwal added a comment - With the patch, the job link on history server UI does not work. Can you try out Siddharth ?
        Hide
        Siddharth Seth added a comment -

        Fixes the UI issue and some unit test failures.
        Finish time is being set on TaskCompletion.

        Show
        Siddharth Seth added a comment - Fixes the UI issue and some unit test failures. Finish time is being set on TaskCompletion.
        Hide
        Sharad Agarwal added a comment -

        I committed to MR-279 branch. Thanks Siddharth!

        Show
        Sharad Agarwal added a comment - I committed to MR-279 branch. Thanks Siddharth!

          People

          • Assignee:
            Siddharth Seth
            Reporter:
            Siddharth Seth
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development