[ 
https://issues.apache.org/jira/browse/DRILL-5429?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15993813#comment-15993813
 ] 

ASF GitHub Bot commented on DRILL-5429:
---------------------------------------

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

    https://github.com/apache/drill/pull/817#discussion_r114429261
  
    --- Diff: 
contrib/format-maprdb/src/main/java/org/apache/drill/exec/store/mapr/db/json/JsonTableGroupScan.java
 ---
    @@ -100,30 +104,46 @@ public GroupScan clone(List<SchemaPath> columns) {
         return newScan;
       }
     
    +  public JsonTableGroupScan clone(JsonScanSpec scanSpec) {
    +    JsonTableGroupScan newScan = new JsonTableGroupScan(this);
    +    newScan.scanSpec = scanSpec;
    +    newScan.computeRegionsToScan();
    +    return newScan;
    +  }
    +
    --- End diff --
    
    Please add comments describing the function


> Improve query performance for MapR DB JSON Tables
> -------------------------------------------------
>
>                 Key: DRILL-5429
>                 URL: https://issues.apache.org/jira/browse/DRILL-5429
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Query Planning & Optimization, Storage - MapRDB
>    Affects Versions: 1.10.0
>            Reporter: Padma Penumarthy
>            Assignee: Padma Penumarthy
>             Fix For: 1.11.0
>
>
> For MapR DB JSON Tables, cache (per query) and reuse table and tabletInfo, 
> instead of fetching the same information multiple times from DB server.  
> Also, getting tableStats is an expensive operation.  We can avoid doing that 
> and instead, get total rowCount from tabletInfo instead.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to