[
https://issues.apache.org/jira/browse/CALCITE-3980?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Haisheng Yuan resolved CALCITE-3980.
------------------------------------
Fix Version/s: 1.23.0
Resolution: Fixed
Fixed in
[https://github.com/apache/calcite/commit/09b13cf6c61739d3ea15adf80b9dc0bc2b1ad0a0],
thanks for the PR, [~neoremind]!
> 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
> Fix For: 1.23.0
>
> 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)
