[jira] [Commented] (CALCITE-3584) Support SQL hints propagation for decorrelation

["Julian Hyde (Jira)" <ji...@apache.org>]

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

Julian Hyde commented on CALCITE-3584:
--------------------------------------

* Rename {{RelNode.setHints}} to {{withHints}}? "set" implies that it mutates the {{RelNode}}, which it does not.
* Which means that {{Hintable.attachHints}} delegates all interesting work to {{withHints}} and {{mergeHints}}. Maybe make {{attachHints}} final, or obsolete it?
* While you're in there, change javadoc "Attach list of hints to this relational expression" to "Attaches list of hints to this relational expression" and so forth.

> Support SQL hints propagation for decorrelation
> -----------------------------------------------
>
>                 Key: CALCITE-3584
>                 URL: https://issues.apache.org/jira/browse/CALCITE-3584
>             Project: Calcite
>          Issue Type: Sub-task
>          Components: core
>    Affects Versions: 1.21.0
>            Reporter: Danny Chen
>            Assignee: Danny Chen
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.22.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> In CALCITE-482, we have support a basic SQL hints framework to let other engines to extend and have their own hints implementations. But for the decorrelation, the hints would just lost.
> We should support that because most of the sub-queries needs decorrelation.



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