[CALCITE-3870] Change the default value of SqlToRelConverter.Config.expand from true to false

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.34.0
• Component/s: core
• Labels: pull-request-available
• Assignee: Julian Hyde
• Reporter: Jiatao Tao
• Created: 25/Mar/20 03:48
• Updated: 14/Mar/23 13:53
• Resolved: 19/Feb/23 20:37

Description

Set "SqlToRelConverter.ConfigBuilder#expand" default to false. Now the default "expand" in  SqlToRelConverter.ConfigBuilder is true but in calcite's main process, actually, it is false( withExpand(THREAD_EXPAND.get()))
 
That leads we need to explicitly set withExpand to false when we use SqlToRelConverter.
 
So I think we should change the default to "false" in SqlToRelConverter.ConfigBuilder.
 

Issue Links

• is related to: CALCITE-1045 Decorrelate sub-queries in Project and Join
• is related to: CALCITE-5127 Support correlation variables in Project
• links to: GitHub Pull Request #1871
• links to: GitHub Pull Request #3065