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

ASF GitHub Bot commented on CARBONDATA-204:
-------------------------------------------

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

    https://github.com/apache/incubator-carbondata/pull/123#discussion_r77506707
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/carbon/querystatistics/DriverQueryStatisticsRecorder.java
 ---
    @@ -78,106 +83,148 @@ public synchronized void 
recordStatisticsForDriver(QueryStatistic statistic, Str
        */
       public void logStatisticsAsTableDriver() {
         synchronized (lock) {
    -      String tableInfo = collectDriverStatistics();
    -      if (null != tableInfo) {
    -        LOGGER.statistic(tableInfo);
    +      Iterator<Map.Entry<String, List<QueryStatistic>>> entries =
    +              queryStatisticsMap.entrySet().iterator();
    +      while (entries.hasNext()) {
    +        Map.Entry<String, List<QueryStatistic>> entry = entries.next();
    +        String queryId = entry.getKey();
    +        // clear the unknown query statistics
    +        if(StringUtils.isEmpty(queryId)) {
    +          queryStatisticsMap.remove(queryId);
    --- End diff --
    
    I tried this, but caused some exceptions


> Query statistics issue 
> -----------------------
>
>                 Key: CARBONDATA-204
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-204
>             Project: CarbonData
>          Issue Type: Bug
>            Reporter: Gin-zhj
>            Assignee: Gin-zhj
>
> Query statistics issue:
> 1. some query statistics that never be printed will be keeped into 
> querystatisticsMap, so it will cause "out of memory" for long time running
> 2. in some sceniaro, the driver can't record "sql_parse_time" , the driver 
> statistics logs will not be output



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to