[ 
https://issues.apache.org/jira/browse/KYLIN-4063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Yaqian Zhang reassigned KYLIN-4063:
-----------------------------------

    Assignee: Yaqian Zhang

> Avoid repeatedly calling "string.toLowerCase" in TimedJsonStreamParser#parse
> ----------------------------------------------------------------------------
>
>                 Key: KYLIN-4063
>                 URL: https://issues.apache.org/jira/browse/KYLIN-4063
>             Project: Kylin
>          Issue Type: Improvement
>          Components: NRT Streaming
>            Reporter: Shaofeng SHI
>            Assignee: Yaqian Zhang
>            Priority: Major
>
> In TimedJsonStreamParser#parse, it has this:
>  
> {code:java}
> for (TblColRef column : allColumns) {
>     final String columnName = column.getName().toLowerCase(Locale.ROOT);
>     if (populateDerivedTimeColumns(columnName, result, t) == false) {
>         result.add(getValueByKey(column, root));
>     }
> }
> {code}
>  
> As this method will be invoked for each message, and then for each column it 
> will have a "toLowerCase(Locale.ROOT)", which is unnecessary, because the 
> "allColumns" won't change.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to