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

Add SET SESSION and RESET statement

    Details

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

      Description

      So far, there have been no session statements which enable users to set or reset session variables. We have only meta command in tsql. Here is my proposal for session statement.

      SET CATALOG and SET TIME ZONE are based on ANSI SQL 2003 standards, and others are based on PostgreSQL. Vertica and Teradata have the same syntax.

      BNF
      : SET CATALOG dbname = identifier
        | SET TIME ZONE (Character_String_Literal | signed_numerical_literal | DEFAULT)
        | SET (SESSION)? name=identifier (TO | EQUAL)?
          (Character_String_Literal | signed_numerical_literal | boolean_literal | DEFAULT)
        | RESET name=identifier
      
      1. TAJO-1238_2.patch
        71 kB
        Hyunsik Choi
      2. TAJO-1238_3.patch
        56 kB
        Hyunsik Choi
      3. TAJO-1238_4.patch
        55 kB
        Hyunsik Choi
      4. TAJO-1238.patch
        57 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/294

        TAJO-1238: Add SET SESSION and RESET statement.

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

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

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

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


        commit 021c872fbfedf90e5eb579dfe109590242fa1bae
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-12-09T01:00:24Z

        Add session statement to parser.

        commit 56b3016a579896e542a44289e59cc7cb769f8880
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-12-09T09:09:27Z

        TAJO-1238: Add SET SESSION and RESET statement.

        commit 46a99857a5adc7b30e532b6dd861157be8262d90
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-12-09T09:09:38Z

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


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user hyunsik opened a pull request: https://github.com/apache/tajo/pull/294 TAJO-1238 : Add SET SESSION and RESET statement. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hyunsik/tajo TAJO-1238 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/294.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 #294 commit 021c872fbfedf90e5eb579dfe109590242fa1bae Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-12-09T01:00:24Z Add session statement to parser. commit 56b3016a579896e542a44289e59cc7cb769f8880 Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-12-09T09:09:27Z TAJO-1238 : Add SET SESSION and RESET statement. commit 46a99857a5adc7b30e532b6dd861157be8262d90 Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-12-09T09:09:38Z Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/tajo into TAJO-1238
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jinossy commented on the pull request:

        https://github.com/apache/tajo/pull/294#issuecomment-66399315

        @hyunsik
        Could you fix the compilation error ?
        ```
        [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile (default-compile) on project tajo-catalog-common: Compilation failure
        [ERROR] /Users/kimjh/asf-patch/incubator-tajo/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java:[57,47] cannot find symbol
        [ERROR] symbol : method isAlphabetic(char)
        [ERROR] location: class java.lang.Character
        ```

        Show
        githubbot ASF GitHub Bot added a comment - Github user jinossy commented on the pull request: https://github.com/apache/tajo/pull/294#issuecomment-66399315 @hyunsik Could you fix the compilation error ? ``` [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.5.1:compile (default-compile) on project tajo-catalog-common: Compilation failure [ERROR] /Users/kimjh/asf-patch/incubator-tajo/tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java: [57,47] cannot find symbol [ERROR] symbol : method isAlphabetic(char) [ERROR] location: class java.lang.Character ```
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/294#issuecomment-66405440

        I missed checking the compilation in JVM 6. I've updated the patch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/294#issuecomment-66405440 I missed checking the compilation in JVM 6. I've updated the patch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/294#issuecomment-66418909

        I fixed some unit test failures.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/294#issuecomment-66418909 I fixed some unit test failures.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jinossy commented on the pull request:

        https://github.com/apache/tajo/pull/294#issuecomment-66420984

        +1 looks good to me

        Show
        githubbot ASF GitHub Bot added a comment - Github user jinossy commented on the pull request: https://github.com/apache/tajo/pull/294#issuecomment-66420984 +1 looks good to me
        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/12686221/TAJO-1238_4.patch
        against master revision release-0.9.0-rc0-82-gbebc780.

        +1 @author. The patch does not contain any @author tags.

        +1 tests included. The patch appears to include 26 new or modified test files.

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

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

        +1 core tests. The patch passed unit tests in tajo-algebra tajo-catalog/tajo-catalog-common tajo-client tajo-common tajo-core tajo-plan.

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/539//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-common.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-client.html
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/539//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/12686221/TAJO-1238_4.patch against master revision release-0.9.0-rc0-82-gbebc780. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 26 new or modified test files. +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 246 new Findbugs (version 2.0.3) warnings. -1 release audit. The applied patch generated 248 release audit warnings. +1 core tests. The patch passed unit tests in tajo-algebra tajo-catalog/tajo-catalog-common tajo-client tajo-common tajo-core tajo-plan. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/539//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-common.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/539//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-client.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/539//console This message is automatically generated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        SUCCESS: Integrated in Tajo-master-build #493 (See https://builds.apache.org/job/Tajo-master-build/493/)
        TAJO-1238: Add SET SESSION and RESET statement. (hyunsik: rev 4bcd46427131c09bed9898a552efe416bbdbf442)

        • tajo-plan/src/main/java/org/apache/tajo/plan/visitor/LogicalPlanVisitor.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog1.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/algebra/AlgebraVisitor.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession4.result
        • tajo-common/src/main/java/org/apache/tajo/util/ProtoUtil.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetSessionNode.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone2.sql
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession5.sql
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/SetSession.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java
        • tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/visitor/BasicLogicalPlanVisitor.java
        • tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLLexer.g4
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone1.sql
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession4.sql
        • tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession5.result
        • tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession3.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/NodeType.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/algebra/BaseAlgebraVisitor.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession1.sql
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession6.sql
        • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
        • tajo-common/src/main/java/org/apache/tajo/util/KeyValueSet.java
        • tajo-client/src/main/java/org/apache/tajo/client/QueryClientImpl.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession3.sql
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone2.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog2.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone3.sql
        • tajo-client/src/main/java/org/apache/tajo/client/QueryClient.java
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
        • tajo-core/src/test/java/org/apache/tajo/engine/query/TestSetSessionQuery.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone3.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession2.sql
        • tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession7.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession7.sql
        • tajo-client/src/main/proto/TajoMasterClientProtocol.proto
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog2.sql
        • tajo-client/src/main/proto/ClientProtos.proto
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/OpType.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession6.result
        • CHANGES
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession1.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession2.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog1.sql
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone1.result
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #493 (See https://builds.apache.org/job/Tajo-master-build/493/ ) TAJO-1238 : Add SET SESSION and RESET statement. (hyunsik: rev 4bcd46427131c09bed9898a552efe416bbdbf442) tajo-plan/src/main/java/org/apache/tajo/plan/visitor/LogicalPlanVisitor.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog1.result tajo-plan/src/main/java/org/apache/tajo/plan/algebra/AlgebraVisitor.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession4.result tajo-common/src/main/java/org/apache/tajo/util/ProtoUtil.java tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetSessionNode.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone2.sql tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession5.sql tajo-algebra/src/main/java/org/apache/tajo/algebra/SetSession.java tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java tajo-plan/src/main/java/org/apache/tajo/plan/visitor/BasicLogicalPlanVisitor.java tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLLexer.g4 tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone1.sql tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession4.sql tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession5.result tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession3.result tajo-plan/src/main/java/org/apache/tajo/plan/logical/NodeType.java tajo-plan/src/main/java/org/apache/tajo/plan/algebra/BaseAlgebraVisitor.java tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession1.sql tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession6.sql tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java tajo-common/src/main/java/org/apache/tajo/util/KeyValueSet.java tajo-client/src/main/java/org/apache/tajo/client/QueryClientImpl.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession3.sql tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone2.result tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog2.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone3.sql tajo-client/src/main/java/org/apache/tajo/client/QueryClient.java tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4 tajo-core/src/test/java/org/apache/tajo/engine/query/TestSetSessionQuery.java tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone3.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession2.sql tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession7.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession7.sql tajo-client/src/main/proto/TajoMasterClientProtocol.proto tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog2.sql tajo-client/src/main/proto/ClientProtos.proto tajo-algebra/src/main/java/org/apache/tajo/algebra/OpType.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession6.result CHANGES tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession1.result tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession2.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog1.sql tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone1.result
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-CODEGEN-build #134 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/134/)
        TAJO-1238: Add SET SESSION and RESET statement. (hyunsik: rev 4bcd46427131c09bed9898a552efe416bbdbf442)

        • tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone3.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java
        • tajo-common/src/main/java/org/apache/tajo/util/ProtoUtil.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession6.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog1.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/visitor/BasicLogicalPlanVisitor.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession4.sql
        • tajo-common/src/main/java/org/apache/tajo/util/KeyValueSet.java
        • tajo-client/src/main/proto/TajoMasterClientProtocol.proto
        • tajo-plan/src/main/java/org/apache/tajo/plan/visitor/LogicalPlanVisitor.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog1.result
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession2.result
        • tajo-client/src/main/java/org/apache/tajo/client/QueryClient.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession2.sql
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone1.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone2.sql
        • tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java
        • tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java
        • CHANGES
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession4.result
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone2.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog2.sql
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession7.result
        • tajo-core/src/test/java/org/apache/tajo/engine/query/TestSetSessionQuery.java
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLLexer.g4
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/SetSession.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession5.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/NodeType.java
        • tajo-client/src/main/java/org/apache/tajo/client/QueryClientImpl.java
        • tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/algebra/BaseAlgebraVisitor.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone3.result
        • tajo-client/src/main/proto/ClientProtos.proto
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession5.result
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession1.sql
        • tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession1.result
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/OpType.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession6.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetSessionNode.java
        • tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java
        • tajo-plan/src/main/java/org/apache/tajo/plan/algebra/AlgebraVisitor.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone1.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog2.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession7.sql
        • tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession3.sql
        • tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession3.result
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #134 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/134/ ) TAJO-1238 : Add SET SESSION and RESET statement. (hyunsik: rev 4bcd46427131c09bed9898a552efe416bbdbf442) tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone3.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanPreprocessor.java tajo-common/src/main/java/org/apache/tajo/util/ProtoUtil.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession6.sql tajo-plan/src/main/java/org/apache/tajo/plan/LogicalPlanner.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog1.sql tajo-plan/src/main/java/org/apache/tajo/plan/visitor/BasicLogicalPlanVisitor.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession4.sql tajo-common/src/main/java/org/apache/tajo/util/KeyValueSet.java tajo-client/src/main/proto/TajoMasterClientProtocol.proto tajo-plan/src/main/java/org/apache/tajo/plan/visitor/LogicalPlanVisitor.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog1.result tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession2.result tajo-client/src/main/java/org/apache/tajo/client/QueryClient.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession2.sql tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone1.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone2.sql tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java tajo-client/src/main/java/org/apache/tajo/client/SessionConnection.java CHANGES tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession4.result tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone2.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/setcatalog2.sql tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession7.result tajo-core/src/test/java/org/apache/tajo/engine/query/TestSetSessionQuery.java tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLLexer.g4 tajo-algebra/src/main/java/org/apache/tajo/algebra/SetSession.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession5.sql tajo-plan/src/main/java/org/apache/tajo/plan/logical/NodeType.java tajo-client/src/main/java/org/apache/tajo/client/QueryClientImpl.java tajo-core/src/main/java/org/apache/tajo/master/GlobalEngine.java tajo-plan/src/main/java/org/apache/tajo/plan/algebra/BaseAlgebraVisitor.java tajo-core/src/test/resources/results/TestSQLAnalyzer/settimezone3.result tajo-client/src/main/proto/ClientProtos.proto tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession5.result tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession1.sql tajo-core/src/main/java/org/apache/tajo/master/TajoMasterClientService.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession1.result tajo-algebra/src/main/java/org/apache/tajo/algebra/OpType.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession6.result tajo-plan/src/main/java/org/apache/tajo/plan/logical/SetSessionNode.java tajo-catalog/tajo-catalog-common/src/main/java/org/apache/tajo/catalog/CatalogUtil.java tajo-plan/src/main/java/org/apache/tajo/plan/algebra/AlgebraVisitor.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/settimezone1.sql tajo-plan/src/main/java/org/apache/tajo/plan/util/PlannerUtil.java tajo-core/src/test/resources/results/TestSQLAnalyzer/setcatalog2.result tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession7.sql tajo-core/src/test/resources/queries/TestSQLAnalyzer/setsession3.sql tajo-core/src/test/resources/results/TestSQLAnalyzer/setsession3.result tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4

          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