crepererum commented on PR #12189: URL: https://github.com/apache/datafusion/pull/12189#issuecomment-2314829484
I think we have roughly the following paths forward here: - just fix this issue here (w/ or w/o regression test) - do NOT require `FusedStream`: we could invest in some tests that patch the physical plan to replace all streams with ones that panic if they are polled after the end (e.g. enforce NOT `FusedStream`) - update the entire stack to require `FusedStream` (requires additional flags/state in a few places, so potentially costly) -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
