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

Tez UI history url handler injects spurious trailing slash

    Details

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

      Description

      When using the following:

          conf.set(TezConfiguration.TEZ_AM_TEZ_UI_HISTORY_URL_TEMPLATE,
           "__HISTORY_URL_BASE__?viewPath=tezapp/__APPLICATION_ID__");
          conf.set(TezConfiguration.TEZ_HISTORY_URL_BASE, "http://localhost/ui/tez");
      

      Final url is "http://localhost/ui/tez/?viewPath=tez-app/TEST_APP_ID" instead of "http://localhost/ui/tez?viewPath=tez-app/TEST_APP_ID"

        Activity

        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.
        Hide
        hitesh Hitesh Shah added a comment -

        Committed to master and branch 0.6

        Show
        hitesh Hitesh Shah added a comment - Committed to master and branch 0.6
        Hide
        hitesh Hitesh Shah added a comment - - edited

        Known findbugs issue. Committing shortly. Thanks for the review Prakash Ramachandran

        Show
        hitesh Hitesh Shah added a comment - - edited Known findbugs issue. Committing shortly. Thanks for the review Prakash Ramachandran
        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/12698614/TEZ-2092.1.patch
        against master revision b74bab4.

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

        +1 tests included. The patch appears to include 1 new or modified test files.

        +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 appears to introduce 1 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/186//testReport/
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TEZ-Build/186//artifact/patchprocess/newPatchFindbugsWarningstez-dag.html
        Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/186//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/12698614/TEZ-2092.1.patch against master revision b74bab4. +1 @author . The patch does not contain any @author tags. +1 tests included . The patch appears to include 1 new or modified test files. +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 appears to introduce 1 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/186//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-TEZ-Build/186//artifact/patchprocess/newPatchFindbugsWarningstez-dag.html Console output: https://builds.apache.org/job/PreCommit-TEZ-Build/186//console This message is automatically generated.
        Hide
        pramachandran Prakash Ramachandran added a comment -

        +1 lgtm

        Show
        pramachandran Prakash Ramachandran added a comment - +1 lgtm
        Hide
        hitesh Hitesh Shah added a comment -

        Prakash Ramachandran Please review.

        Show
        hitesh Hitesh Shah added a comment - Prakash Ramachandran Please review.

          People

          • Assignee:
            hitesh Hitesh Shah
            Reporter:
            hitesh Hitesh Shah
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development