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

Forward Xu commented on CALCITE-3980:
-------------------------------------

well, I will pay attention to these issues.

> Improvement for creating RedisTable 
> ------------------------------------
>
>                 Key: CALCITE-3980
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3980
>             Project: Calcite
>          Issue Type: Improvement
>          Components: csv-adapter
>            Reporter: neoremind
>            Priority: Minor
>
> When creating RedisTable via ModelHandler, it relies on an instance method 
> _RedisEnumerator#deduceRowType_ to extract field type map, redis connection 
> will be established and closed every time a table is created, which is 
> unnecessary and redundant, so simply make the method a static one will solve 
> the problem and make it more performant. Actually, CsvTable implementation 
> works the right way.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to