Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 0.7.1, 0.8.2
    • None
    • None

    Description

      • Tooltips are not disappearing at times
      • Make search message readable
      • Remove Application Attempt Id row from Tez Application Description in app details page.

      Attachments

        1. TEZ-2553.1.patch
          2 kB
          Sreenath Somarajapuram
        2. TEZ-2553.2.patch
          2 kB
          Sreenath Somarajapuram

        Activity

          dbhowmick

          Please review the patch.

          Sreenath Sreenath Somarajapuram added a comment - dbhowmick Please review the patch.
          tezqa TezQA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12769487/TEZ-2553.1.patch
          against master revision 65f4575.

          +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 3.0.1) warnings.

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

          -1 core tests. The patch failed these unit tests in :
          org.apache.tez.dag.app.TestPreemption

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

          This message is automatically generated.

          tezqa TezQA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12769487/TEZ-2553.1.patch against master revision 65f4575. +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 3.0.1) warnings. +1 release audit . The applied patch does not increase the total number of release audit warnings. -1 core tests . The patch failed these unit tests in : org.apache.tez.dag.app.TestPreemption Test results: https://builds.apache.org/job/PreCommit-TEZ-Build/1260//testReport/ Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/1260//console This message is automatically generated.
          hitesh Hitesh Shah added a comment -

          "Substring search ..." is not really more readable. The user does not get clarity on how the search box is meant to be used. Maybe just change to a simple "Search ..." for now.

          hitesh Hitesh Shah added a comment - "Substring search ..." is not really more readable. The user does not get clarity on how the search box is meant to be used. Maybe just change to a simple "Search ..." for now.
          hitesh Hitesh Shah added a comment -

          +1 to commit after the search message is changed.

          hitesh Hitesh Shah added a comment - +1 to commit after the search message is changed.

          Attaching patch with the text change.
          Committing shortly.

          Sreenath Sreenath Somarajapuram added a comment - Attaching patch with the text change. Committing shortly.
          tezqa TezQA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12769601/TEZ-2553.2.patch
          against master revision dcc5108.

          +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 3.0.1) 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/1263//testReport/
          Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/1263//console

          This message is automatically generated.

          tezqa TezQA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12769601/TEZ-2553.2.patch against master revision dcc5108. +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 3.0.1) 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/1263//testReport/ Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/1263//console This message is automatically generated.

          Thank you hitesh
          Committed to both master and branch-0.7.

          Sreenath Sreenath Somarajapuram added a comment - Thank you hitesh Committed to both master and branch-0.7.
          hitesh Hitesh Shah added a comment -

          Closing as 0.7.1 released.

          hitesh Hitesh Shah added a comment - Closing as 0.7.1 released.

          People

            Sreenath Sreenath Somarajapuram
            Sreenath Sreenath Somarajapuram
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: