You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@calcite.apache.org by "Haisheng Yuan (Jira)" <ji...@apache.org> on 2020/05/10 15:48:00 UTC

[jira] [Updated] (CALCITE-3980) Improvement for creating RedisTable

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

Haisheng Yuan updated CALCITE-3980:
-----------------------------------
    Component/s:     (was: csv-adapter)

> Improvement for creating RedisTable 
> ------------------------------------
>
>                 Key: CALCITE-3980
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3980
>             Project: Calcite
>          Issue Type: Improvement
>            Reporter: neoremind
>            Priority: Minor
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> 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)