Details

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

      Description

      The index feature is under development in the index_support branch.
      After TAJO-838, we can create an index and use it for more efficient query execution.
      For more tests and feedbacks, we need to encourage users to use indexes by adding documentations.

        Activity

        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-CODEGEN-build #188 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/188/)
        TAJO-1294: Add index documents. (jihoon) (jihoonson: rev 3308aab4b64d36b805b0b0e6cc282a6dce97f6ad)

        • CHANGES
        • tajo-docs/src/main/sphinx/index_overview.rst
        • tajo-docs/src/main/sphinx/sql_language/ddl.rst
        • tajo-docs/src/main/sphinx/index/how_to_use.rst
        • tajo-docs/src/main/sphinx/index/types.rst
        • tajo-docs/src/main/sphinx/index/future_work.rst
        • tajo-docs/src/main/sphinx/index.rst
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-CODEGEN-build #188 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/188/ ) TAJO-1294 : Add index documents. (jihoon) (jihoonson: rev 3308aab4b64d36b805b0b0e6cc282a6dce97f6ad) CHANGES tajo-docs/src/main/sphinx/index_overview.rst tajo-docs/src/main/sphinx/sql_language/ddl.rst tajo-docs/src/main/sphinx/index/how_to_use.rst tajo-docs/src/main/sphinx/index/types.rst tajo-docs/src/main/sphinx/index/future_work.rst tajo-docs/src/main/sphinx/index.rst
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #549 (See https://builds.apache.org/job/Tajo-master-build/549/)
        TAJO-1294: Add index documents. (jihoon) (jihoonson: rev 3308aab4b64d36b805b0b0e6cc282a6dce97f6ad)

        • CHANGES
        • tajo-docs/src/main/sphinx/index/future_work.rst
        • tajo-docs/src/main/sphinx/sql_language/ddl.rst
        • tajo-docs/src/main/sphinx/index/how_to_use.rst
        • tajo-docs/src/main/sphinx/index.rst
        • tajo-docs/src/main/sphinx/index_overview.rst
        • tajo-docs/src/main/sphinx/index/types.rst
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #549 (See https://builds.apache.org/job/Tajo-master-build/549/ ) TAJO-1294 : Add index documents. (jihoon) (jihoonson: rev 3308aab4b64d36b805b0b0e6cc282a6dce97f6ad) CHANGES tajo-docs/src/main/sphinx/index/future_work.rst tajo-docs/src/main/sphinx/sql_language/ddl.rst tajo-docs/src/main/sphinx/index/how_to_use.rst tajo-docs/src/main/sphinx/index.rst tajo-docs/src/main/sphinx/index_overview.rst tajo-docs/src/main/sphinx/index/types.rst
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson closed the pull request at:

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

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

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/344#issuecomment-69447255

        @hyunsik thank you for the review!
        I have committed it after fixing the compilation warning.

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/344#issuecomment-69447255 @hyunsik thank you for the review! I have committed it after fixing the compilation warning.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/344#issuecomment-69446947

        +1 Ship it. The patch looks good to me. Could you fix the compilation warning before committing it?

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/344#issuecomment-69446947 +1 Ship it. The patch looks good to me. Could you fix the compilation warning before committing it?
        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/344#discussion_r22757142

        — Diff: tajo-docs/src/main/sphinx/index_overview.rst —
        @@ -0,0 +1,20 @@
        +***********************
        — End diff –

        The asterisk iteration should be longer than the title. Otherwise, it causes warning.

        ```
        sphinx-build -b html -d target/doctrees src/main/sphinx target/html
        Making output directory...
        Running Sphinx v1.2.1
        loading pickled environment... not yet created
        building [html]: targets for 58 source files that are out of date
        updating environment: 58 added, 0 changed, 0 removed
        reading sources... [100%] tsql/variables
        /Users/hyunsik/Code/tajo/tajo-10/tajo-docs/src/main/sphinx/index_overview.rst:1: WARNING: Title overline too short.

        ***********************
        Index (Experimental Feature)
        ***********************
        looking for now-outdated files... none found
        pickling environment... done
        checking consistency... done
        preparing documents... done
        writing output... [100%] tsql/variables
        writing additional files... genindex search
        copying static files... WARNING: html_static_path entry u'/Users/hyunsik/Code/tajo/tajo-10/tajo-docs/src/main/sphinx/static' does not exist
        done
        copying extra files... done
        dumping search index... done
        dumping object inventory... done
        build succeeded, 2 warnings.

        Build finished. The HTML pages are in target/html.
        ```

        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/344#discussion_r22757142 — Diff: tajo-docs/src/main/sphinx/index_overview.rst — @@ -0,0 +1,20 @@ +*********************** — End diff – The asterisk iteration should be longer than the title. Otherwise, it causes warning. ``` sphinx-build -b html -d target/doctrees src/main/sphinx target/html Making output directory... Running Sphinx v1.2.1 loading pickled environment... not yet created building [html] : targets for 58 source files that are out of date updating environment: 58 added, 0 changed, 0 removed reading sources... [100%] tsql/variables /Users/hyunsik/Code/tajo/tajo-10/tajo-docs/src/main/sphinx/index_overview.rst:1: WARNING: Title overline too short. *********************** Index (Experimental Feature) *********************** looking for now-outdated files... none found pickling environment... done checking consistency... done preparing documents... done writing output... [100%] tsql/variables writing additional files... genindex search copying static files... WARNING: html_static_path entry u'/Users/hyunsik/Code/tajo/tajo-10/tajo-docs/src/main/sphinx/static' does not exist done copying extra files... done dumping search index... done dumping object inventory... done build succeeded, 2 warnings. Build finished. The HTML pages are in target/html. ```
        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/12691165/TAJO-1294.patch
        against master revision release-0.9.0-rc0-135-g807868b.

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

        +0 tests included. The patch appears to be a documentation patch that doesn't require tests.

        +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 does not introduce any new Findbugs (version 2.0.3) warnings.

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

        +1 core tests. The patch passed unit tests in .

        Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/572//testReport/
        Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/572//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt
        Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/572//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/12691165/TAJO-1294.patch against master revision release-0.9.0-rc0-135-g807868b. +1 @author. The patch does not contain any @author tags. +0 tests included. The patch appears to be a documentation patch that doesn't require tests. +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 does not introduce any new Findbugs (version 2.0.3) warnings. -1 release audit. The applied patch generated 720 release audit warnings. +1 core tests. The patch passed unit tests in . Test results: https://builds.apache.org/job/PreCommit-TAJO-Build/572//testReport/ Release audit warnings: https://builds.apache.org/job/PreCommit-TAJO-Build/572//artifact/incubator-tajo/patchprocess/patchReleaseAuditProblems.txt Console output: https://builds.apache.org/job/PreCommit-TAJO-Build/572//console This message is automatically generated.
        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user jihoonson opened a pull request:

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

        TAJO-1294: Add index documents

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

        $ git pull https://github.com/jihoonson/tajo-2 TAJO-1294

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

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


        commit eda174682926bcefb31c00b00bea713aa36a393b
        Author: Jihoon Son <jihoonson@apache.org>
        Date: 2015-01-09T15:05:21Z

        TAJO-1294

        commit 53ef56726226c4eb03b9934cabf5a681e8f20a2e
        Author: Jihoon Son <jihoonson@apache.org>
        Date: 2015-01-09T15:06:55Z

        TAJO-1294


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user jihoonson opened a pull request: https://github.com/apache/tajo/pull/344 TAJO-1294 : Add index documents You can merge this pull request into a Git repository by running: $ git pull https://github.com/jihoonson/tajo-2 TAJO-1294 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/344.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 #344 commit eda174682926bcefb31c00b00bea713aa36a393b Author: Jihoon Son <jihoonson@apache.org> Date: 2015-01-09T15:05:21Z TAJO-1294 commit 53ef56726226c4eb03b9934cabf5a681e8f20a2e Author: Jihoon Son <jihoonson@apache.org> Date: 2015-01-09T15:06:55Z TAJO-1294

          People

          • Assignee:
            jihoonson Jihoon Son
            Reporter:
            jihoonson Jihoon Son
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development