timsaucer commented on issue #21155: URL: https://github.com/apache/datafusion/issues/21155#issuecomment-4129488545
The reason this worked in 51.0.0 is because we created a `SessionContext` within the FFI providers. So when we called `create_physical_plan` in `DataFrameTableProvider` it had a *local* `SessionContext` (not the actual session context that registration was happening with) and that was able to do the appropriate downcasting to `ViewTable` for the table scan. The reason this does not work in 52.3.0 is because we now have a full `FFI_Session` instead of creating a temporary locally held `SessionContext`. Thus when we have a `Arc<dyn TableProvider>` in the protobuf serialization code it shows up as a `ForeignTableProvider` and NOT a `ViewTable`. Thus we try the "fall back" to a logical codec which gives the user error we end up with. -- 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]
