alamb commented on issue #7836: URL: https://github.com/apache/arrow-datafusion/issues/7836#issuecomment-1767422838
Hi @amartins23 It seems like the consumer functions such as https://docs.rs/datafusion-substrait/latest/datafusion_substrait/logical_plan/consumer/fn.from_substrait_plan.html https://docs.rs/datafusion-substrait/latest/datafusion_substrait/physical_plan/consumer/fn.from_substrait_rel.html Require mutable access, but the producers do not (they already take `&SessionContext`, not `&mut SessionContext`) https://docs.rs/datafusion-substrait/latest/datafusion_substrait/logical_plan/producer/fn.to_substrait_plan.html This I think this ticket would require: 1. Converting from &mut SessionContext` to &SessionContext` 2. Ideally add some doc comments / examples of how to do this conversion This seems straightforward and a good first project so marking it as such Note there may be some subtlety I don't understand that would make this not feasible. -- 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]
