zabetak commented on pull request #2690: URL: https://github.com/apache/calcite/pull/2690#issuecomment-1014688116
It's true that the `RelBuilder` API around `RepeatUnion` is tight to a table (it may not even be a `TransientTable` but let's not focus on this right now) but the javadoc of `RepeatUnion` and its implementation (`EnumerableRepeatUnion`) do not mention at all tables, spools, or other operators. If you want to change the contract maybe it would be better to send an email to the dev list to see what people think and if there are others using `RepeatUnion` in some other way. Apart from that, it might worth brainstorming a bit if the problem could be solved by handling the addition/removal of transient tables at another level and not during the execution of some specific operator; I don't have ideas at the moment :laughing: -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
