rdettai commented on pull request #1072:
URL: https://github.com/apache/arrow-datafusion/pull/1072#issuecomment-936866927
> Could you elaborate on the root cause for requiring this static?
The problem is mainly present in Ballista:
- you need the `ObjectStore` in the `ExecutionPlan` and the `TableProvider`
- both these need to be serialized and deserialized (either in the scheduler
or the executor)
- how do we ensure that the right object store are instantiated in the
various components?
- the object store implementation might be in a different crate, so we
need to make sure that all component where compiled with the right dependencies
- what does it even mean to serialize an object store?
- referring to one common instance of the registry from all the different
places helps this
I am not really happy about this solution either, so any alternative
solution is welcome!
--
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]
