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

HCatalogStore need to be deprecated

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.10.0, 0.10.1
    • Fix Version/s: 0.11.0, 0.12.0
    • Component/s: Catalog
    • Labels:
      None

      Description

      If old version users use tajo 0.11.0 using existing hcatalog configuration, they will fail to run TajoMaster with ClassNotFoundException. So, we need to deprecate HCatalogStore.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user blrunner opened a pull request:

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

        TAJO-1775: HCatalogStore need to be deprecated.

        If old version users use tajo 0.11.0 using existing hcatalog configuration, they will fail to run TajoMaster with ClassNotFoundException. So, we need to deprecate HCatalogStore.

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

        $ git pull https://github.com/blrunner/tajo TAJO-1775

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

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


        commit 8598e8cf58c1d20914d75e5e65649cd7eb498abb
        Author: JaeHwa Jung <blrunner@apache.org>
        Date: 2015-08-17T03:24:24Z

        TAJO-1775: HCatalogStore need to be deprecated.


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user blrunner opened a pull request: https://github.com/apache/tajo/pull/694 TAJO-1775 : HCatalogStore need to be deprecated. If old version users use tajo 0.11.0 using existing hcatalog configuration, they will fail to run TajoMaster with ClassNotFoundException. So, we need to deprecate HCatalogStore. You can merge this pull request into a Git repository by running: $ git pull https://github.com/blrunner/tajo TAJO-1775 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/694.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 #694 commit 8598e8cf58c1d20914d75e5e65649cd7eb498abb Author: JaeHwa Jung <blrunner@apache.org> Date: 2015-08-17T03:24:24Z TAJO-1775 : HCatalogStore need to be deprecated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/694#issuecomment-131908647

        I think that the unit test seems to be not necessary because it is striaghtforward.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/694#issuecomment-131908647 I think that the unit test seems to be not necessary because it is striaghtforward.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/694#issuecomment-131989244

        Thanks @hyunsik
        I've removed it.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/694#issuecomment-131989244 Thanks @hyunsik I've removed it.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/694#issuecomment-132213511

        Looks good to me.

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/694#issuecomment-132213511 Looks good to me.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/694#issuecomment-132502686

        +1

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/694#issuecomment-132502686 +1
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        FAILURE: Integrated in Tajo-master-CODEGEN-build #446 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/446/)
        TAJO-1775: HCatalogStore need to be deprecated. (blrunner: rev 2a850cfc5adbf48f982b63a79e6b759d16ef425b)

        • tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java
          TAJO-1775: HCatalogStore need to be deprecated. (missing change) (blrunner: rev 9ddda140587e83f849bbd78d7cb79d31952df641)
        • CHANGES
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #446 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/446/ ) TAJO-1775 : HCatalogStore need to be deprecated. (blrunner: rev 2a850cfc5adbf48f982b63a79e6b759d16ef425b) tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java TAJO-1775 : HCatalogStore need to be deprecated. (missing change) (blrunner: rev 9ddda140587e83f849bbd78d7cb79d31952df641) CHANGES
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-build #807 (See https://builds.apache.org/job/Tajo-master-build/807/)
        TAJO-1775: HCatalogStore need to be deprecated. (blrunner: rev 2a850cfc5adbf48f982b63a79e6b759d16ef425b)

        • tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java
          TAJO-1775: HCatalogStore need to be deprecated. (missing change) (blrunner: rev 9ddda140587e83f849bbd78d7cb79d31952df641)
        • CHANGES
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-build #807 (See https://builds.apache.org/job/Tajo-master-build/807/ ) TAJO-1775 : HCatalogStore need to be deprecated. (blrunner: rev 2a850cfc5adbf48f982b63a79e6b759d16ef425b) tajo-catalog/tajo-catalog-drivers/tajo-hive/src/main/java/org/apache/tajo/catalog/store/HCatalogStore.java TAJO-1775 : HCatalogStore need to be deprecated. (missing change) (blrunner: rev 9ddda140587e83f849bbd78d7cb79d31952df641) CHANGES

          People

          • Assignee:
            blrunner Jaehwa Jung
            Reporter:
            blrunner Jaehwa Jung
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development