alexandreyc opened a new pull request, #1742:
URL: https://github.com/apache/arrow-adbc/pull/1742

   Hey!
   
   Here is the second PR of the Rust implementation containing full FFI 
bindings.
   
   I've decided to manually implement bindings instead of relying on 
[bindgen](https://rust-lang.github.io/rust-bindgen/) because:
   - We don't want an additional dependency on clang (which bindgen depends on) 
and so we cannot generate bindings at compile-time
   - We could have checked-in the generated bindings in git but in that case 
manually editing is unavoidable because generated bindings are 
platform-dependent
   - Besides, we still have to implement default methods, conversion traits and 
release mechanisms.
   
   In the end, I found it simpler to have everything manually implemented 
rather than having some parts automatic and some others manual... 
[arrow-rs](https://github.com/apache/arrow-rs) seems to have followed the same 
path.
   
   CC @mbrobbel @paleolimbot @aljazerzen 


-- 
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]

Reply via email to