[ 
https://issues.apache.org/jira/browse/CALCITE-3219?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16907211#comment-16907211
 ] 

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
(v7.6.14#76016)

Reply via email to