You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@calcite.apache.org by GitBox <gi...@apache.org> on 2022/01/15 17:53:30 UTC

[GitHub] [calcite] zabetak commented on pull request #2690: RepeatUnion improvements: CALCITE-3673 + CALCITE-4054

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: commits-unsubscribe@calcite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org