Pressenna commented on CALCITE-3219:

created https://github.com/apache/calcite/pull/1378 for the changes, which I 
would use as a discussion basis.

> Allow finer control over which expressions get expanded in SqlToRelConverter
> ----------------------------------------------------------------------------
>                 Key: CALCITE-3219
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3219
>             Project: Calcite
>          Issue Type: Improvement
>            Reporter: Pressenna
>            Priority: Major
>              Labels: pull-request-available
>          Time Spent: 10m
>  Remaining Estimate: 0h
> 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.
> {code}
> // 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()
> {code}
> If this is OK, I would provide a PR with the relevant changes.

This message was sent by Atlassian JIRA

Reply via email to