julianhyde commented on issue #1457: [WIP][Prototype][CALCITE-3340] TUMBLE as a TableFunction URL: https://github.com/apache/calcite/pull/1457#issuecomment-535647198 I take your point that with the limited namespace we can't have two different functions called TUMBLE. Therefore changing the Parser.jj seems to make sense. However, we see the future as being TUMBLE (or any user-defined table function) in the FROM clause. Therefore how about adding parser support for TUMBLE (and HOP) in the GROUP BY clause? This would be deprecated functionality, and removed at some point. The parser would be hard-wired to create calls to that function, and therefore function does not need to be called literally TUMBLE. It could be called "TUMBLE_OLD" or whatever.
---------------------------------------------------------------- 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
