amaliujia edited a comment on issue #1587: [CALCITE-3272] Support TUMBLE as 
Table Valued Function including an enumerable implementation, stream.iq and 
DESCRIPTOR
URL: https://github.com/apache/calcite/pull/1587#issuecomment-568384386
 
 
   Gotcha. So you think the better way is to use `SqlKind.OTHER_FUNCTION`? It's 
fine to me as I don't know what's the better solution for that. Let know if you 
find something and we can see how to update the code.
   
   `Descriptor` is supposed to have the same scope as the TVF for sure. I think 
that's what I am doing this PR: build the scope from the first operand of TVF, 
then use that scope for `Descriptor`.
   
   I will try to add more tests first then. That should be easier to do.
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to