You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Sean Broeder (Jira)" <ji...@apache.org> on 2020/08/24 18:57:00 UTC

[jira] [Reopened] (CALCITE-4183) FilterSetOpTransposeRule constructor should allow for user defined Filter and SetOp classes

     [ https://issues.apache.org/jira/browse/CALCITE-4183?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sean Broeder reopened CALCITE-4183:
-----------------------------------

Reopening so that changes can be reviewed

> FilterSetOpTransposeRule constructor should allow for user defined Filter and SetOp classes
> -------------------------------------------------------------------------------------------
>
>                 Key: CALCITE-4183
>                 URL: https://issues.apache.org/jira/browse/CALCITE-4183
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.24.0
>            Reporter: Sean Broeder
>            Priority: Major
>   Original Estimate: 168h
>          Time Spent: 10m
>  Remaining Estimate: 168h
>
> The current FilterSetOpTransposeRule constructor simply takes a RelBuilderFactory.  Adding an additional constructor would allow more flexibility for user specific Filter and SetOp classes and follows behavior already allowed for several other rules.  
>  For example:
> /**
>  * Creates a FilterSetOpTransposeRule that uses a generic
>  * {@link Filter} and {@link SetOp}
>  * @param filterClass filter class
>  * @param setOpClass setOp class
>  * @param relBuilderFactory builder factory for relational expressions
>  */
> public FilterSetOpTransposeRule(Class<? extends Filter> filterClass,
>                                 Class<? extends SetOp> setOpClass,
>                                 RelBuilderFactory relBuilderFactory) {
>    super(
>        operand(filterClass,
>        operand(setOpClass, any())),
>        relBuilderFactory, null);
> }



--
This message was sent by Atlassian Jira
(v8.3.4#803005)