rdettai commented on pull request #1107: URL: https://github.com/apache/arrow-datafusion/pull/1107#issuecomment-941235005
Interesting binding! I have done a few refacto these past weeks that modified the `ExecutionContext` API. My felling is that it is not 100% stable yet. Those changes required me to modify the python bindings, which added an overhead of ~5-10% to my work. If we add bindings for many other languages, this will be a non negligible burden on contributors to the core datafusion/ballista Rust code 🦀. I don't have a binding vote and I am biased by the fact that I struggled with this recently, but my _personal_ opinion is that we should consider leaving language bindings in external crates, at least for now. 😃 -- 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]
