Details

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

      Description

      Pleasae see the discussion.
      http://markmail.org/message/avo2jj2jbys4dasn

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user jinossy opened a pull request:

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

        TAJO-1129: Remove hadoop 2.2.0 support

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

        $ git pull https://github.com/jinossy/tajo TAJO-1129

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

        https://github.com/apache/tajo/pull/215.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 #215


        commit b2d09ef6c81a300f67e1bddb9d14e4ad8c0fa7fe
        Author: jhkim <jhkim@apache.org>
        Date: 2014-10-27T09:18:53Z

        TAJO-1129: Remove hadoop 2.2.0 support


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user jinossy opened a pull request: https://github.com/apache/tajo/pull/215 TAJO-1129 : Remove hadoop 2.2.0 support You can merge this pull request into a Git repository by running: $ git pull https://github.com/jinossy/tajo TAJO-1129 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/215.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 #215 commit b2d09ef6c81a300f67e1bddb9d14e4ad8c0fa7fe Author: jhkim <jhkim@apache.org> Date: 2014-10-27T09:18:53Z TAJO-1129 : Remove hadoop 2.2.0 support
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/215#issuecomment-60625404

        +1
        It's just changes in documentation.

        Aside from this issue, are there a way to enforce Hadoop version? It's because some guys may give an unsupported Hadoop version when they build Tajo through maven.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/215#issuecomment-60625404 +1 It's just changes in documentation. Aside from this issue, are there a way to enforce Hadoop version? It's because some guys may give an unsupported Hadoop version when they build Tajo through maven.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jinossy commented on the pull request:

        https://github.com/apache/tajo/pull/215#issuecomment-61488617

        @hyunsik
        We should consider the runtime error and CDH hadoop. because ZCR interface is included the 2.3.0 higher.
        How about checking the interface HasEnhancedByteBufferAccess ?
        https://issues.apache.org/jira/browse/HDFS-4953

        Show
        githubbot ASF GitHub Bot added a comment - Github user jinossy commented on the pull request: https://github.com/apache/tajo/pull/215#issuecomment-61488617 @hyunsik We should consider the runtime error and CDH hadoop. because ZCR interface is included the 2.3.0 higher. How about checking the interface HasEnhancedByteBufferAccess ? https://issues.apache.org/jira/browse/HDFS-4953
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/215#issuecomment-61568901

        I meant that some users may input ```-Dhadoop.version=2.2.0``` to maven builds. In this case, the compilation errors will occur during maven build. It would be great if we prohibit wrong hadoop versions.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/215#issuecomment-61568901 I meant that some users may input ```-Dhadoop.version=2.2.0``` to maven builds. In this case, the compilation errors will occur during maven build. It would be great if we prohibit wrong hadoop versions.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/215#issuecomment-62286142

        I investigated the way to enforce the version of Hadoop. Maven enforce seems to do it. But, it requires custom maven plugin using maven enforcer. So, I think that my suggestion should be dealt by another jira issue.

        Here is my +1. Please ship this documentation change.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/215#issuecomment-62286142 I investigated the way to enforce the version of Hadoop. Maven enforce seems to do it. But, it requires custom maven plugin using maven enforcer. So, I think that my suggestion should be dealt by another jira issue. Here is my +1. Please ship this documentation change.
        Hide
        hyunsik Hyunsik Choi added a comment -

        I just committed the patch to master branch. Thank you Jinho for your work.

        Show
        hyunsik Hyunsik Choi added a comment - I just committed the patch to master branch. Thank you Jinho for your work.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #437 (See https://builds.apache.org/job/Tajo-master-build/437/)
        TAJO-1129: Remove hadoop 2.2.0 support. (jinho via hyunsik) (hyunsik: rev 5c82b7db42c718652d2a1fbb7bdd29d852e9565d)

        • README
        • BUILDING
        • tajo-docs/src/main/sphinx/getting_started/prerequisites.rst
        • CHANGES
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #437 (See https://builds.apache.org/job/Tajo-master-build/437/ ) TAJO-1129 : Remove hadoop 2.2.0 support. (jinho via hyunsik) (hyunsik: rev 5c82b7db42c718652d2a1fbb7bdd29d852e9565d) README BUILDING tajo-docs/src/main/sphinx/getting_started/prerequisites.rst CHANGES
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #79 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/79/)
        TAJO-1129: Remove hadoop 2.2.0 support. (jinho via hyunsik) (hyunsik: rev 5c82b7db42c718652d2a1fbb7bdd29d852e9565d)

        • README
        • BUILDING
        • CHANGES
        • tajo-docs/src/main/sphinx/getting_started/prerequisites.rst
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #79 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/79/ ) TAJO-1129 : Remove hadoop 2.2.0 support. (jinho via hyunsik) (hyunsik: rev 5c82b7db42c718652d2a1fbb7bdd29d852e9565d) README BUILDING CHANGES tajo-docs/src/main/sphinx/getting_started/prerequisites.rst
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development