[
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)
