[CALCITE-3980] Redis-adapter redis connection is not reused when RedisTable is created

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.23.0
• Assignee: Unassigned
• Reporter: neoremind
• Created: 08/May/20 10:30
• Updated: 24/May/20 12:00
• Resolved: 10/May/20 15:54

Description

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.