alamb commented on issue #150: URL: https://github.com/apache/arrow-datafusion/issues/150#issuecomment-953837996
> @alamb if I understand correctly, the issue is looking at making this MemoryStream pub. But there is also another reference under Ballista for the same here, I assume that this is not related to the issue. That is a good find @1nF0rmed -- yes we are talking about making https://github.com/apache/arrow-datafusion/blob/295033492bbf335cdc02446cefb3751ab840f9b5/datafusion/src/physical_plan/memory.rs#L161-L171 public. the fact that ballista has a copy/paste version of the same code is likely due to the fact that it is not `pub` lol -- so perhaps as part of the PR that makes `MemoryStream` pub we can remove the duplicate in Ballista -- 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]
