[
https://issues.apache.org/jira/browse/CALCITE-6574?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17881130#comment-17881130
]
James Starr commented on CALCITE-6574:
--------------------------------------
[~julianhyde]I believe several of the helper functions and factory methods are
called during scope population, so that solution would not work. My initial
thoughts would be to have a context that is propagated in place of the
SqlValidatorImpl, the context would initially be populated with the initial
helper functions and of such. Then just expand the functions that type
validation to include it in their method signatures. Then on a per method
bases, the logic can inverted to in type validator or the scope populator
depend on the context of the method. However, what ever strategy is selected
for breaking up the dependency of the scopes/namespaces on the validator impl,
it would be easier or at least not more difficult if I continued with my
initial purposed changes of splitting the the scope populating and type
validation into explicit classes?
> Add ScopeMap
> ------------
>
> Key: CALCITE-6574
> URL: https://issues.apache.org/jira/browse/CALCITE-6574
> Project: Calcite
> Issue Type: Sub-task
> Reporter: James Starr
> Assignee: James Starr
> Priority: Major
> Labels: pull-request-available
>
> See parent description.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)