ctsk commented on PR #15431: URL: https://github.com/apache/datafusion/pull/15431#issuecomment-2769391782
I'm moving this back to the draft stage, because it will conflict with https://github.com/apache/datafusion/pull/15476 once I fix that PR. I can't see a way to achieve the goals of this PR and https://github.com/apache/datafusion/pull/15476 at the same time with a single OnceCell. I suspect that if I pursue #15476, I can not avoid storing a SendableRecordBatchStream in the ExecutionPlan. So overall, it seems to me like this PR is less clear of a win. -- 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
