[CALCITE-485] Avoid recreate/reparse of CalciteConnectionConfigImpl in CalciteConnectionImpl.config()

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Affects Version/s: 1.0.0-incubating
• Labels: newbie
• Assignee: Unassigned
• Reporter: Vladimir Sitnikov
• Created: 28/Nov/14 08:00
• Updated: 22/Feb/19 01:22

Description

CalciteConnectionImpl.config is used in lots of places, however the resulting CalciteConnectionConfig is immutable enough, so:
1) No need to recreate new config object every time
2) No need to "parse" the values of input map (e.g. like CalciteConnectionProperty.QUOTING.wrap(properties).getEnum(Quoting.class, lex().quoting)).

I think we can just pre-parse all the values in the CalciteConnectionConfig constructor and store the results to fields.

Issue Links

• links to: Dev branch