alamb commented on pull request #1072: URL: https://github.com/apache/arrow-datafusion/pull/1072#issuecomment-937962030
I like this enumeration of possibilities (1) and (2). 👍 I think solution (2) is also a reasonable approach. Another thought I had if we want to do (1) without a static `ObjectStoreRegistry` may be to deserialize an `ObjectStore` reference to a `PlaceholderObjectStore` and then add a subsequent `resolve_object_store` pass over the deserialied `ExecutionPlans` to replace all `PlaceholderObjectStore` instances with an instance of the correct type. Not the cleanest solution, but it would avoid trying to plumb something into the serde serialization -- 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]
