You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by spanchamiamapr <gi...@git.apache.org> on 2016/12/05 19:00:28 UTC

[GitHub] drill pull request #677: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

GitHub user spanchamiamapr opened a pull request:

    https://github.com/apache/drill/pull/677

    DRILL-5065 - Optimize count( * ) queries on MapR-DB JSON Tables

    In MapR-DB v5.2.0, we enabled '_id' only projection for JSON tables.
    Hence, we can now optimize the following queries:
    a. count(*) by projecting only the '_id' column.
    b. '_id' only projections, including count(_id)
    
    Change the format plugin config parameter name.

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

    $ git pull https://github.com/spanchamiamapr/drill drill-5065

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

    https://github.com/apache/drill/pull/677.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 #677
    
----
commit 0c07e2857f09e6f262b246e795f10901ed744e3d
Author: Smidth Panchamia <sp...@mapr.com>
Date:   2016-11-28T21:59:34Z

    DRILL-5065 - Optimize count( * ) queries on MapR-DB JSON Tables
    In MapR-DB v5.2.0, we enabled '_id' only projection for JSON tables.
    Hence, we can now optimize the following queries:
    a. count(*) by projecting only the '_id' column.
    b. '_id' only projections, including count(_id)
    
    Change the format plugin config parameter name.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request #677: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

Posted by spanchamiamapr <gi...@git.apache.org>.
Github user spanchamiamapr closed the pull request at:

    https://github.com/apache/drill/pull/677


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

[GitHub] drill pull request #677: DRILL-5065 - Optimize count( * ) queries on MapR-DB...

Posted by adityakishore <gi...@git.apache.org>.
Github user adityakishore commented on a diff in the pull request:

    https://github.com/apache/drill/pull/677#discussion_r90932924
  
    --- Diff: contrib/format-maprdb/src/main/java/org/apache/drill/exec/store/mapr/db/MapRDBFormatPluginConfig.java ---
    @@ -65,6 +68,15 @@ public void setAllTextMode(boolean mode) {
         allTextMode = mode;
       }
     
    +  @JsonProperty("disableCountOptimization")
    +  public void setReadDocumentForCount(boolean mode) {
    --- End diff --
    
    setter function does not match the parameter.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---