zabetak commented on pull request #2690: URL: https://github.com/apache/calcite/pull/2690#issuecomment-1013724967
Hey @rubenada , I don't have much time the following 1-2 weeks to dive into this :/ I had a quick look but cannot have a clear picture in 10-15 min. I am afraid that putting the burden of adding/removing the transient table to the `EnumerableRepeatUnion` operator will make the latter less generic since it implies that we will not be able to use it with other sub-plans. About CALCITE-3673, can't we remove the table while close the enumerator of the `ListTransientTable`? -- 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]
