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

Showing query progress percentage in TAJO JDBC

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Trivial
    • Resolution: Won't Fix
    • Affects Version/s: 0.11.1
    • Fix Version/s: 0.11.1
    • Component/s: JDBC Driver
    • Labels:
      None

      Description

      It would be nice to check out query progress percentage in TAJO JDBC like TAJO CLI already did. The purpose is simple. I want to verify whether query is in block or not by monitoring query progress percentage.

      While TAJO CLI uses display formatter, it would be nice to provide standard logging system in TAJO JDBC. That would be more convenient and make sense to let end users choose debugging level.

      It would be helpful to print log messages such as an method below.
      org.apache.tajo.client.TajoClientUtil.waitCompletion(QueryClient, QueryId)

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user combineads opened a pull request:

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

        TAJO-2029: Showing query progress percentage in TAJO JDBC

        It would be nice to check out query progress percentage in TAJO JDBC like TAJO CLI already did. The purpose is simple. I want to verify whether query is in block or not by monitoring query progress percentage.

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

        $ git pull https://github.com/combineads/tajo TAJO-2029

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

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


        commit d3ffcabb44539427532da20678c788e867098800
        Author: Byunghwa Yun <combine@combineads.co.kr>
        Date: 2015-12-16T05:26:27Z

        Remove unused import class

        commit 31637c41cdcebbccc5854de8a131e5cd86f1c4a3
        Author: combineads <combine@combineads.co.kr>
        Date: 2015-12-16T05:57:01Z

        Merge remote-tracking branch 'upstream/master'

        commit 463dd93c0dcc0b5f022308ad2adc3da7068c4441
        Author: combineads <combine@combineads.co.kr>
        Date: 2015-12-18T02:35:32Z

        Merge remote-tracking branch 'upstream/master'

        commit 516cc981fe2b6946eea47c2e6ec93be625d7098e
        Author: Byunghwa Yun <combine@combineads.co.kr>
        Date: 2015-12-18T03:02:02Z

        Showing query progress percentage in TAJO JDBC


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user combineads opened a pull request: https://github.com/apache/tajo/pull/917 TAJO-2029 : Showing query progress percentage in TAJO JDBC It would be nice to check out query progress percentage in TAJO JDBC like TAJO CLI already did. The purpose is simple. I want to verify whether query is in block or not by monitoring query progress percentage. You can merge this pull request into a Git repository by running: $ git pull https://github.com/combineads/tajo TAJO-2029 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/917.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 #917 commit d3ffcabb44539427532da20678c788e867098800 Author: Byunghwa Yun <combine@combineads.co.kr> Date: 2015-12-16T05:26:27Z Remove unused import class commit 31637c41cdcebbccc5854de8a131e5cd86f1c4a3 Author: combineads <combine@combineads.co.kr> Date: 2015-12-16T05:57:01Z Merge remote-tracking branch 'upstream/master' commit 463dd93c0dcc0b5f022308ad2adc3da7068c4441 Author: combineads <combine@combineads.co.kr> Date: 2015-12-18T02:35:32Z Merge remote-tracking branch 'upstream/master' commit 516cc981fe2b6946eea47c2e6ec93be625d7098e Author: Byunghwa Yun <combine@combineads.co.kr> Date: 2015-12-18T03:02:02Z Showing query progress percentage in TAJO JDBC
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/917#issuecomment-169260472

        Hi @combineads, thanks for your patch.
        However, I'm sorry to say but this patch looks a sort of workaround to see the query progress. I think we need to devise a better solution.

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/917#issuecomment-169260472 Hi @combineads, thanks for your patch. However, I'm sorry to say but this patch looks a sort of workaround to see the query progress. I think we need to devise a better solution.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user combineads commented on the pull request:

        https://github.com/apache/tajo/pull/917#issuecomment-169268140

        I just need to check the execution progress of query.

        The issue would be close?

        Thanks for response @jihoonson .

        Show
        githubbot ASF GitHub Bot added a comment - Github user combineads commented on the pull request: https://github.com/apache/tajo/pull/917#issuecomment-169268140 I just need to check the execution progress of query. The issue would be close? Thanks for response @jihoonson .
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/917#issuecomment-169273497

        I appreciate if you do. Thanks.

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/917#issuecomment-169273497 I appreciate if you do. Thanks.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user combineads closed the pull request at:

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

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

        Thanks Byunghwa Yun. Even though we close your PR, I think that this feature looks still useful. We can reopen later when it comes to being required.

        Show
        jihoonson Jihoon Son added a comment - Thanks Byunghwa Yun . Even though we close your PR, I think that this feature looks still useful. We can reopen later when it comes to being required.

          People

          • Assignee:
            combine Byunghwa Yun
            Reporter:
            combine Byunghwa Yun
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development