Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-575

Worker's env.jsp has wrong URL which go to worker's index.jsp.

        Details

        • Type: Bug
        • Status: Resolved
        • Priority: Trivial
        • Resolution: Fixed
        • Affects Version/s: 0.8.0
        • Fix Version/s: 0.8.0
        • Component/s: Web UI
        • Labels:
          None

          Description

          "ndex.jsp" must be changed to "index.jsp"

          1. Text File
            TAJO-575.patch
            0.7 kB
            Hyoungjun Kim

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            hjkim Hyoungjun Kim added a comment -

            I modified "ndex.jsp" to "index.jps" in wokrer/env.jsp.
            Please review this patch.

            Show
            hjkim Hyoungjun Kim added a comment - I modified "ndex.jsp" to "index.jps" in wokrer/env.jsp. Please review this patch.
            Hide
            Permalink
            tajoqa Tajo QA added a comment -

            -1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12626592/TAJO-575.patch
            against master revision 10c599f.

            +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. The applied patch does not increase the total number of javadoc warnings.

            +1 checkstyle. The patch generated 0 code style errors.

            -1 findbugs. The patch appears to introduce 192 new Findbugs (version 1.3.9) 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 tajo-core/tajo-core-backend.

            Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/97//testReport/
            Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/97//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-backend.html
            Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/97//console

            This message is automatically generated.

            Show
            tajoqa Tajo QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12626592/TAJO-575.patch against master revision 10c599f. +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. The applied patch does not increase the total number of javadoc warnings. +1 checkstyle. The patch generated 0 code style errors. -1 findbugs. The patch appears to introduce 192 new Findbugs (version 1.3.9) 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 tajo-core/tajo-core-backend. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/97//testReport/ Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/97//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core-backend.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/97//console This message is automatically generated.
            Hide
            Permalink
            jihoonson Jihoon Son added a comment -

            +1

            Show
            jihoonson Jihoon Son added a comment - +1
            Hide
            Permalink
            blrunner Jaehwa Jung added a comment -

            +1.

            I committed it to the master branch.

            Show
            blrunner Jaehwa Jung added a comment - +1. I committed it to the master branch.
            Hide
            Permalink
            hudson Hudson added a comment -

            SUCCESS: Integrated in Tajo-master-build #72 (See https://builds.apache.org/job/Tajo-master-build/72/)
            TAJO-575: Worker's env.jsp has wrong URL which go to worker's index.jsp. (hyoungjunkim via jaehwa) (jhjung: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=ccc2bd0658b107fb08817d730070d58b8be88a0f)

            • CHANGES.txt
            • tajo-core/tajo-core-backend/src/main/resources/webapps/worker/env.jsp
            Show
            hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #72 (See https://builds.apache.org/job/Tajo-master-build/72/ ) TAJO-575 : Worker's env.jsp has wrong URL which go to worker's index.jsp. (hyoungjunkim via jaehwa) (jhjung: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=ccc2bd0658b107fb08817d730070d58b8be88a0f ) CHANGES.txt tajo-core/tajo-core-backend/src/main/resources/webapps/worker/env.jsp

              People

              • Assignee:
                hjkim Hyoungjun Kim
                Reporter:
                hjkim Hyoungjun Kim
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development