nevi-me commented on pull request #7297: URL: https://github.com/apache/arrow/pull/7297#issuecomment-642223515
@andygrove @nealrichardson Rust tests are now running (binary found). The problem was that we were setting the build output directory to `build/rust/target/...` instead of the default `arrow/rust/target/...` folder. Then in the integration tests we were looking for the Rust binary in the latter location. I've removed the part of the build script in `ci/scripts/rust_build.sh` that did the above, and unit tests are also still running correctly. Should I go ahead and remove the lines that I commented out? Neal, I'm not sure of what else would need to be done for the integration tests to run on each commit/PR. Otherwise I think this is good to merge. ---------------------------------------------------------------- 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: [email protected]
