Uploaded image for project: 'Apache Tez'
  1. Apache Tez
  2. TEZ-2165

Tez UI: DAG shows running status if killed by RM in some cases.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.6.1
    • Component/s: UI
    • Labels:
      None
    • Target Version/s:

      Description

      TEZ-2038 had fixed the issue of DAG showing running status if killed, if the info is available from AHS
      due to a typo in javascript code, in some cases it still shows running.
      eg. yarnAppState is FINISHED but yarnAppFinalState is KILLED.

      1. TEZ-2165.1.patch
        0.5 kB
        Prakash Ramachandran

        Activity

        Hide
        pramachandran Prakash Ramachandran added a comment -

        trivial patch to fix typo

        Show
        pramachandran Prakash Ramachandran added a comment - trivial patch to fix typo
        Hide
        hitesh Hitesh Shah added a comment -

        +1.

        Show
        hitesh Hitesh Shah added a comment - +1.
        Hide
        hadoopqa Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12701590/TEZ-2165.1.patch
        against master revision 0a81409.

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no new tests are needed for this patch.
        Also please list what manual steps were performed to verify this patch.

        +1 javac. The applied patch does not increase the total number of javac compiler warnings.

        +1 javadoc. There were no new javadoc warning messages.

        +1 findbugs. The patch does not introduce any new Findbugs (version 2.0.3) warnings.

        +1 release audit. The applied patch does not increase the total number of release audit warnings.

        +1 core tests. The patch passed unit tests in .

        Test results: https://builds.apache.org/job/PreCommit-TEZ-Build/238//testReport/
        Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/238//console

        This message is automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12701590/TEZ-2165.1.patch against master revision 0a81409. +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javac . The applied patch does not increase the total number of javac compiler warnings. +1 javadoc . There were no new javadoc warning messages. +1 findbugs . The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. +1 core tests . The patch passed unit tests in . Test results: https://builds.apache.org/job/PreCommit-TEZ-Build/238//testReport/ Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/238//console This message is automatically generated.
        Hide
        hitesh Hitesh Shah added a comment -

        COmmitting shortly.

        Show
        hitesh Hitesh Shah added a comment - COmmitting shortly.
        Hide
        hitesh Hitesh Shah added a comment -

        Thanks Prakash Ramachandran. Committed. Mind opening a general jira for adding unit tests for all such helpers?

        Show
        hitesh Hitesh Shah added a comment - Thanks Prakash Ramachandran . Committed. Mind opening a general jira for adding unit tests for all such helpers?
        Hide
        hitesh Hitesh Shah added a comment -

        Closing issue as 0.5.4, 0.6.1 and 0.7.0 have been released.

        Show
        hitesh Hitesh Shah added a comment - Closing issue as 0.5.4, 0.6.1 and 0.7.0 have been released.

          People

          • Assignee:
            pramachandran Prakash Ramachandran
            Reporter:
            pramachandran Prakash Ramachandran
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development