You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/04/22 13:30:53 UTC

[GitHub] [arrow] jorgecarleitao commented on a change in pull request #10096: ARROW-12444: [Rust] Remove rust

jorgecarleitao commented on a change in pull request #10096:
URL: https://github.com/apache/arrow/pull/10096#discussion_r618402563



##########
File path: dev/archery/archery/lang/rust.py
##########
@@ -1,23 +0,0 @@
-# Licensed to the Apache Software Foundation (ASF) under one

Review comment:
       AFAIK this was only used to call `Cargo fmt` from Python; the build of the integration binaries is done by `rust_build.sh`; Python calls those binaries without `Cargo`.
   
   I also checking [the logs](https://github.com/apache/arrow/pull/10096/checks?check_run_id=2377696454), we can find prints such as
   
   ```
   ##########################################################
   IPC: C++ producing, Rust consuming
   ##########################################################
   ```
   
   indicating that Rust is being tested.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org