alamb commented on PR #10259: URL: https://github.com/apache/datafusion/pull/10259#issuecomment-2083079688
> Hence, I think it is better to proceed with current approach in this PR. (as in prefer_existing_union) Sounds good to me. > In the future, if we add support for OrderPreservingInterleaveExec (this might be accomplished by replacing CombinedRecordBatchStream with streaming_merge in the fn execute method of the InterleaveExec.) This is an excellent idea, I will file a ticket to track the idea -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
