Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.0, 0.11.0
    • Component/s: SQL Parser
    • Labels:
      None

      Description

      I plan to implement ALTER TABLE ADD/DROP PARTITION statement stage by stage. For the first time, I'll add the statement to parser.

      Syntax
      ALTER TABLE table_name [IF NOT EXISTS] ADD PARTITION (key1 = 'val2', key2 = 'val2', ...) [LOCATION 'partition_path_of_directory'];
      ALTER TABLE table_name [IF NOT EXISTS] DROP PARTITION (key1 = 'val2', key2 = 'val2', ...)
      
      1. TAJO-1260_2.patch
        40 kB
        Jaehwa Jung
      2. TAJO-1260_3.patch
        40 kB
        Jaehwa Jung

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user blrunner opened a pull request:

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

        TAJO-1260: Add ALTER TABLE ADD/DROP PARTITION statement to parser

        I plan to implement ALTER TABLE ADD/DROP PARTITION statement stage by stage. For the first time, I'll add the statement to parser.

        • Syntax

        ALTER TABLE table_name [IF NOT EXISTS] ADD PARTITION (key1 = 'val2', key2 = 'val2', ...) [LOCATION 'partition_path_of_directory'];

        ALTER TABLE table_name [IF NOT EXISTS] DROP PARTITION (key1 = 'val2', key2 = 'val2', ...);

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

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

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

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


        commit ead3c99954ba417852085221bd3b2a2680893be5
        Author: JaeHwa Jung <blrunner@apache.org>
        Date: 2015-01-07T07:14:34Z

        TAJO-1260: Add ALTER TABLE ADD/DROP PARTITION statement to parser


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user blrunner opened a pull request: https://github.com/apache/tajo/pull/335 TAJO-1260 : Add ALTER TABLE ADD/DROP PARTITION statement to parser I plan to implement ALTER TABLE ADD/DROP PARTITION statement stage by stage. For the first time, I'll add the statement to parser. Syntax ALTER TABLE table_name [IF NOT EXISTS] ADD PARTITION (key1 = 'val2', key2 = 'val2', ...) [LOCATION 'partition_path_of_directory'] ; ALTER TABLE table_name [IF NOT EXISTS] DROP PARTITION (key1 = 'val2', key2 = 'val2', ...); You can merge this pull request into a Git repository by running: $ git pull https://github.com/blrunner/tajo TAJO-1260 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/335.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 #335 commit ead3c99954ba417852085221bd3b2a2680893be5 Author: JaeHwa Jung <blrunner@apache.org> Date: 2015-01-07T07:14:34Z TAJO-1260 : Add ALTER TABLE ADD/DROP PARTITION statement to parser
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on a diff in the pull request:

        https://github.com/apache/tajo/pull/335#discussion_r22575086

        — Diff: tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java —
        @@ -367,6 +364,59 @@ public void testCreateTablePartitionByColumn() throws IOException {
        }

        @Test
        + public void testAlterTableAddPartition1() throws IOException {
        + String sql = FileUtil.readTextFile(new File("src/test/resources/queries/default/alter_table_add_partition_1.sql"));
        — End diff –

        could you use FileUtil.readTextFileFromResource instead of direct paths? Direct path can cause errors without setting a base directory.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on a diff in the pull request: https://github.com/apache/tajo/pull/335#discussion_r22575086 — Diff: tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java — @@ -367,6 +364,59 @@ public void testCreateTablePartitionByColumn() throws IOException { } @Test + public void testAlterTableAddPartition1() throws IOException { + String sql = FileUtil.readTextFile(new File("src/test/resources/queries/default/alter_table_add_partition_1.sql")); — End diff – could you use FileUtil.readTextFileFromResource instead of direct paths? Direct path can cause errors without setting a base directory.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-68992733

        Hi @hyunsik

        Thanks for your quick review. I used FileUtil.readTextFileFromResource instead of direct paths and I added more unit test cases.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-68992733 Hi @hyunsik Thanks for your quick review. I used FileUtil.readTextFileFromResource instead of direct paths and I added more unit test cases.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-69169946

        Could you trigger the travisCI job?

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-69169946 Could you trigger the travisCI job?
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-69270517

        Hi @hyunsik

        I've just merged the master branch to TAJO-1260 branch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-69270517 Hi @hyunsik I've just merged the master branch to TAJO-1260 branch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-69288694

        +1
        The patch looks good to me.

        BTW, you need to consider inequality expression in DROP PARTITION. But, the current patch is good starting point. We can improve it later in another jira.
        ```sql
        ALTER TABLE table_name [IF EXISTS] DROP PARTITION (key1 [=|<|<=|>|>=|!=] 'val1', key2 ...,);
        ```

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-69288694 +1 The patch looks good to me. BTW, you need to consider inequality expression in DROP PARTITION. But, the current patch is good starting point. We can improve it later in another jira. ```sql ALTER TABLE table_name [IF EXISTS] DROP PARTITION (key1 [=|<|<=|>|>=|!=] 'val1', key2 ...,); ```
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-69447746

        ``ALTER TABLE .. [ADD | DROP] PARTITION`` statement will cause error if this statement is given in JDBC driver or tsql. So, could you add some exception handling code to prevent ugly errors?

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-69447746 ``ALTER TABLE .. [ADD | DROP] PARTITION`` statement will cause error if this statement is given in JDBC driver or tsql. So, could you add some exception handling code to prevent ugly errors?
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-71165638

        Hi @hyunsik

        I added the exception handling code. If possible, could you check the pathch again?

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-71165638 Hi @hyunsik I added the exception handling code. If possible, could you check the pathch again?
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-71201071

        Sure, I'll review it soon.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-71201071 Sure, I'll review it soon.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-72601449

        @blrunner 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/335#issuecomment-72601449 @blrunner Could you trigger TravisCI?
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user blrunner commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-72773906

        @hyunsik

        Sure, I've just triggered it.

        Show
        githubbot ASF GitHub Bot added a comment - Github user blrunner commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-72773906 @hyunsik Sure, I've just triggered it.
        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/12696389/TAJO-1260.patch
        against master revision release-0.9.0-rc0-167-gae7862b.

        -1 patch. The patch command could not apply the patch.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/582//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/12696389/TAJO-1260.patch against master revision release-0.9.0-rc0-167-gae7862b. -1 patch. The patch command could not apply the patch. -1 patch. The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/582//console This message is automatically generated.
        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/12696392/TAJO-1260_2.patch
        against master revision release-0.9.0-rc0-167-gae7862b.

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

        +1 tests included. The patch appears to include 12 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 212 new Findbugs (version 2.0.3) warnings.

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

        -1 core tests. The patch failed these unit tests in tajo-algebra tajo-core tajo-plan:
        org.apache.tajo.jdbc.TestTajoJdbc

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/583//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/583//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/12696392/TAJO-1260_2.patch against master revision release-0.9.0-rc0-167-gae7862b. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 12 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 212 new Findbugs (version 2.0.3) warnings. -1 release audit. The applied patch generated 885 release audit warnings. -1 core tests. The patch failed these unit tests in tajo-algebra tajo-core tajo-plan: org.apache.tajo.jdbc.TestTajoJdbc Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/583//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/583//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/583//console This message is automatically generated.
        Hide
        blrunner Jaehwa Jung added a comment -

        I fixed a build failure.

        Show
        blrunner Jaehwa Jung added a comment - I fixed a build failure.
        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/12696401/TAJO-1260_3.patch
        against master revision release-0.9.0-rc0-167-gae7862b.

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

        +1 tests included. The patch appears to include 12 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 212 new Findbugs (version 2.0.3) warnings.

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

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

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/584//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html
        Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/584//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/12696401/TAJO-1260_3.patch against master revision release-0.9.0-rc0-167-gae7862b. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 12 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 212 new Findbugs (version 2.0.3) warnings. -1 release audit. The applied patch generated 373 release audit warnings. +1 core tests. The patch passed unit tests in tajo-algebra tajo-core tajo-plan. Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/584//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-plan.html Findbugs warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/584//artifact/incubator-tajo/patchprocess/newPatchFindbugsWarningstajo-core.html Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/584//console This message is automatically generated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/335#issuecomment-72958948

        +1 ship it. Thanks.

        It should be committed to only master branch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/335#issuecomment-72958948 +1 ship it. Thanks. It should be committed to only master branch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        Thanks Hyunsik Choi,

        I've just committed it to the master branch.

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

        FAILURE: Integrated in Tajo-master-CODEGEN-build #220 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/220/)
        TAJO-1260: Add ALTER TABLE ADD/DROP PARTITION statement to parser (jaehwa) (blrunner: rev 3f336f500bd53b829467c9c7f6e3fe777554a423)

        • tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTable.java
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_2.sql
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTableOpType.java
        • tajo-core/src/test/java/org/apache/tajo/jdbc/TestTajoJdbc.java
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_1.sql
        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_3.sql
        • tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java
        • tajo-core/src/test/resources/results/TestTajoCli/testAlterTableDropPartition.result
        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_2.sql
        • CHANGES
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_3.sql
        • tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_4.sql
        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_1.sql
        • tajo-core/src/test/resources/results/TestTajoCli/testAlterTableAddPartition.result
        • tajo-core/src/test/java/org/apache/tajo/cli/tsql/TestTajoCli.java
        • tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-CODEGEN-build #220 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/220/ ) TAJO-1260 : Add ALTER TABLE ADD/DROP PARTITION statement to parser (jaehwa) (blrunner: rev 3f336f500bd53b829467c9c7f6e3fe777554a423) tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTable.java tajo-core/src/test/resources/queries/default/alter_table_add_partition_2.sql tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTableOpType.java tajo-core/src/test/java/org/apache/tajo/jdbc/TestTajoJdbc.java tajo-core/src/test/resources/queries/default/alter_table_add_partition_1.sql tajo-core/src/test/resources/queries/default/alter_table_drop_partition_3.sql tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java tajo-core/src/test/resources/results/TestTajoCli/testAlterTableDropPartition.result tajo-core/src/test/resources/queries/default/alter_table_drop_partition_2.sql CHANGES tajo-core/src/test/resources/queries/default/alter_table_add_partition_3.sql tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4 tajo-core/src/test/resources/queries/default/alter_table_add_partition_4.sql tajo-core/src/test/resources/queries/default/alter_table_drop_partition_1.sql tajo-core/src/test/resources/results/TestTajoCli/testAlterTableAddPartition.result tajo-core/src/test/java/org/apache/tajo/cli/tsql/TestTajoCli.java tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-build #581 (See https://builds.apache.org/job/Tajo-master-build/581/)
        TAJO-1260: Add ALTER TABLE ADD/DROP PARTITION statement to parser (jaehwa) (blrunner: rev 3f336f500bd53b829467c9c7f6e3fe777554a423)

        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_3.sql
        • tajo-core/src/test/resources/results/TestTajoCli/testAlterTableAddPartition.result
        • tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTable.java
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_3.sql
        • tajo-core/src/test/resources/results/TestTajoCli/testAlterTableDropPartition.result
        • tajo-core/src/test/java/org/apache/tajo/jdbc/TestTajoJdbc.java
        • CHANGES
        • tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4
        • tajo-core/src/test/java/org/apache/tajo/cli/tsql/TestTajoCli.java
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_2.sql
        • tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTableOpType.java
        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_2.sql
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_1.sql
        • tajo-core/src/test/resources/queries/default/alter_table_drop_partition_1.sql
        • tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java
        • tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java
        • tajo-core/src/test/resources/queries/default/alter_table_add_partition_4.sql
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-build #581 (See https://builds.apache.org/job/Tajo-master-build/581/ ) TAJO-1260 : Add ALTER TABLE ADD/DROP PARTITION statement to parser (jaehwa) (blrunner: rev 3f336f500bd53b829467c9c7f6e3fe777554a423) tajo-core/src/test/resources/queries/default/alter_table_drop_partition_3.sql tajo-core/src/test/resources/results/TestTajoCli/testAlterTableAddPartition.result tajo-plan/src/main/java/org/apache/tajo/plan/verifier/PreLogicalPlanVerifier.java tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTable.java tajo-core/src/test/resources/queries/default/alter_table_add_partition_3.sql tajo-core/src/test/resources/results/TestTajoCli/testAlterTableDropPartition.result tajo-core/src/test/java/org/apache/tajo/jdbc/TestTajoJdbc.java CHANGES tajo-core/src/main/antlr4/org/apache/tajo/engine/parser/SQLParser.g4 tajo-core/src/test/java/org/apache/tajo/cli/tsql/TestTajoCli.java tajo-core/src/test/resources/queries/default/alter_table_add_partition_2.sql tajo-algebra/src/main/java/org/apache/tajo/algebra/AlterTableOpType.java tajo-core/src/test/resources/queries/default/alter_table_drop_partition_2.sql tajo-core/src/test/resources/queries/default/alter_table_add_partition_1.sql tajo-core/src/test/resources/queries/default/alter_table_drop_partition_1.sql tajo-core/src/main/java/org/apache/tajo/engine/parser/SQLAnalyzer.java tajo-core/src/test/java/org/apache/tajo/engine/parser/TestSQLAnalyzer.java tajo-core/src/test/resources/queries/default/alter_table_add_partition_4.sql

          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