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

Fix wrong schema name in JDBC driver

    Details

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

      Description

      This patch fixes wrong default schema name used in JDBC driver. So far, we have used 'public' in a default schema name. This patch will change it to an empty string (i.e., "");

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user blrunner opened a pull request:

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

        TAJO-1225: Fix wrong schema name in JDBC driver

        This patch fixes wrong default schema name used in JDBC driver. So far, we have used 'public' in a default schema name. This patch will change it to an empty string (i.e., "");

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

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

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

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


        commit 80c721c242fcfc66deceb23ab22fb7050570df22
        Author: JaeHwa Jung <blrunner@apache.org>
        Date: 2015-01-06T02:37:31Z

        TAJO-1225: Fix wrong schema name in JDBC driver

        commit edd4b555a3b75f19b8302ce1f5fd7168a6d6a36f
        Author: JaeHwa Jung <blrunner@apache.org>
        Date: 2015-01-06T02:37:54Z

        Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into TAJO-1225


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user blrunner opened a pull request: https://github.com/apache/tajo/pull/330 TAJO-1225 : Fix wrong schema name in JDBC driver This patch fixes wrong default schema name used in JDBC driver. So far, we have used 'public' in a default schema name. This patch will change it to an empty string (i.e., ""); You can merge this pull request into a Git repository by running: $ git pull https://github.com/blrunner/tajo TAJO-1225 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/330.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 #330 commit 80c721c242fcfc66deceb23ab22fb7050570df22 Author: JaeHwa Jung <blrunner@apache.org> Date: 2015-01-06T02:37:31Z TAJO-1225 : Fix wrong schema name in JDBC driver commit edd4b555a3b75f19b8302ce1f5fd7168a6d6a36f Author: JaeHwa Jung <blrunner@apache.org> Date: 2015-01-06T02:37:54Z Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into TAJO-1225
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/330#issuecomment-70051424

        Could you trigger TravisCI?

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/330#issuecomment-70051424 Could you trigger TravisCI?
        Hide
        blrunner Jaehwa Jung added a comment -

        Thanks Hyunsik Choi

        I've just committed the patch to the master branch.

        Show
        blrunner Jaehwa Jung added a comment - Thanks Hyunsik Choi I've just committed the patch to the master branch.
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #556 (See https://builds.apache.org/job/Tajo-master-build/556/)
        TAJO-1225: Fix wrong schema name in JDBC driver. (jaehwa) (blrunner: rev 76ece8b874f4a0c5554e784d7bfcc1474b217c47)

        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables1.result
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas2.result
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables2.result
        • tajo-common/src/main/java/org/apache/tajo/TajoConstants.java
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas1.result
        • CHANGES
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #556 (See https://builds.apache.org/job/Tajo-master-build/556/ ) TAJO-1225 : Fix wrong schema name in JDBC driver. (jaehwa) (blrunner: rev 76ece8b874f4a0c5554e784d7bfcc1474b217c47) tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables1.result tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas2.result tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables2.result tajo-common/src/main/java/org/apache/tajo/TajoConstants.java tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas1.result CHANGES
        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-CODEGEN-build #195 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/195/)
        TAJO-1225: Fix wrong schema name in JDBC driver. (jaehwa) (blrunner: rev 76ece8b874f4a0c5554e784d7bfcc1474b217c47)

        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas1.result
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables2.result
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas2.result
        • tajo-common/src/main/java/org/apache/tajo/TajoConstants.java
        • tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables1.result
        • CHANGES
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-CODEGEN-build #195 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/195/ ) TAJO-1225 : Fix wrong schema name in JDBC driver. (jaehwa) (blrunner: rev 76ece8b874f4a0c5554e784d7bfcc1474b217c47) tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas1.result tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables2.result tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getSchemas2.result tajo-common/src/main/java/org/apache/tajo/TajoConstants.java tajo-core/src/test/resources/results/TestTajoDatabaseMetaData/getTables1.result CHANGES

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development