tustvold edited a comment on pull request #1283: URL: https://github.com/apache/arrow-datafusion/pull/1283#issuecomment-981520642
I did some poking around a while back and was going to write up an issue, but got pulled away to other things. The problem is that SendableRecordBatchStream has a Sync constraint on it. I think this is just a mistake in DataFusion, I can't see a compelling reason that being able to share immutable references to streams across threads is important, especially when stream consumption requires &mut. This would, however, be a fairly annoying breaking change to DataFusion and I wanted to provide more context before filing a ticket, but ran out of time -- 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]
