[CALCITE-3219] Allow finer control over which expressions get expanded in SqlToRelConverter

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Abandoned
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Pressenna
• Created: 30/Jul/19 02:04
• Updated: 03/Sep/20 09:35
• Resolved: 03/Sep/20 09:35

Description

Currently one can control if sub-query expansion happens via SqlToRelConverter.Config.isExpand(). This flag is global to the query and causes expansion of all supported expression types. For some systems it is required that one can dynamically narrow the set of expressions that should be expanded.

In order to allow this, I would propose a new setter with a default implementation that returns the above flag.

// a BiPredicate in order to pass both the original query and the expression in question for expansion
java.util.function.BiPredicate<SqlNode, SqlNode> SqlToRelConverter.Config.getExpandPredicate()

If this is OK, I would provide a PR with the relevant changes.

Issue Links

• is depended upon by: CALCITE-3186 IN expressions in UPDATE statements throws Exceptions
• links to: GitHub Pull Request #1378