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

rename tajo-yarn-pullserver to tajo-pullserver

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.0
    • Component/s: None
    • Labels:
      None

      Description

      'tajo-yarn-pullserver' still has a legacy name. We need to change it to tajo-pullserver.

      1. TAJO-1229.patch
        212 kB
        Hyunsik Choi

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user hyunsik opened a pull request:

        https://github.com/apache/tajo/pull/284

        TAJO-1229: rename tajo-yarn-pullserver to tajo-pullserver.

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/hyunsik/tajo TAJO-1229

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/284.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #284


        commit 09d350c55fd8d805461406c6388fd5fc77795431
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-12-04T16:50:38Z

        TAJO-1229: rename tajo-yarn-pullserver to tajo-pullserver.


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user hyunsik opened a pull request: https://github.com/apache/tajo/pull/284 TAJO-1229 : rename tajo-yarn-pullserver to tajo-pullserver. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hyunsik/tajo TAJO-1229 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/284.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #284 commit 09d350c55fd8d805461406c6388fd5fc77795431 Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-12-04T16:50:38Z TAJO-1229 : rename tajo-yarn-pullserver to tajo-pullserver.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/284#issuecomment-66036223

        +1

        Thanks @hyunsik.
        The patch looks good to me and all unit test cases finished successfully.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/284#issuecomment-66036223 +1 Thanks @hyunsik. The patch looks good to me and all unit test cases finished successfully.
        Hide
        hyunsik Hyunsik Choi added a comment -

        committed.

        Show
        hyunsik Hyunsik Choi added a comment - committed.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/284

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/284
        Hide
        tajoqa Tajo QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12685130/TAJO-1229.patch
        against master revision release-0.9.0-rc0-75-gfacd1dd.

        +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 10 new Findbugs (version 2.0.3) warnings.

        -1 release audit. The applied patch generated 184 release audit warnings.

        +1 core tests. The patch passed unit tests in tajo-pullserver.

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/534//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/534//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/534//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-pullserver.html
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/534//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/12685130/TAJO-1229.patch against master revision release-0.9.0-rc0-75-gfacd1dd. +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 10 new Findbugs (version 2.0.3) warnings. -1 release audit. The applied patch generated 184 release audit warnings. +1 core tests. The patch passed unit tests in tajo-pullserver. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/534//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/534//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/534//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-pullserver.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/534//console This message is automatically generated.
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #127 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/127/)
        TAJO-1229: rename tajo-yarn-pullserver to tajo-pullserver. (hyunsik: rev b5aa780460fcfbf657541ee6c94d41b34b1b24b9)

        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java
        • CHANGES
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java
        • pom.xml
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java
        • tajo-pullserver/pom.xml
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java
        • tajo-yarn-pullserver/pom.xml
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #127 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/127/ ) TAJO-1229 : rename tajo-yarn-pullserver to tajo-pullserver. (hyunsik: rev b5aa780460fcfbf657541ee6c94d41b34b1b24b9) tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java CHANGES tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java pom.xml tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java tajo-pullserver/pom.xml tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java tajo-yarn-pullserver/pom.xml tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #486 (See https://builds.apache.org/job/Tajo-master-build/486/)
        TAJO-1229: rename tajo-yarn-pullserver to tajo-pullserver. (hyunsik: rev b5aa780460fcfbf657541ee6c94d41b34b1b24b9)

        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java
        • tajo-pullserver/pom.xml
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java
        • pom.xml
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java
        • CHANGES
        • tajo-yarn-pullserver/pom.xml
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java
        • tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java
        • tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #486 (See https://builds.apache.org/job/Tajo-master-build/486/ ) TAJO-1229 : rename tajo-yarn-pullserver to tajo-pullserver. (hyunsik: rev b5aa780460fcfbf657541ee6c94d41b34b1b24b9) tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java tajo-pullserver/pom.xml tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerPipelineFactory.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerUtil.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DataRetriever.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FileCloseListener.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/AdvancedDataRetriever.java pom.xml tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServerService.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedFileRegion.java CHANGES tajo-yarn-pullserver/pom.xml tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/FileChunk.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/RetrieverHandler.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/retriever/DirectoryRetriever.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/FileAccessForbiddenException.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpUtil.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/TajoPullServer.java tajo-yarn-pullserver/src/main/java/org/apache/tajo/pullserver/HttpDataServerHandler.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/PullServerAuxService.java tajo-pullserver/src/main/java/org/apache/tajo/pullserver/FadvisedChunkedFile.java

          People

          • Assignee:
            hyunsik Hyunsik Choi
            Reporter:
            hyunsik Hyunsik Choi
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development