mqy edited a comment on pull request #8967:
URL: https://github.com/apache/arrow/pull/8967#issuecomment-749651061
> I would personally be happy to accept this PR (based on calling git or
something else) if simply running cargo test would pass all tests without
having to set the *_TEST_DATA variables ( I think you would just need to make
the changes you suggested of "Existing codes can be updated in this way :")
Actually, in previous commits (that was reverted), I had updated other
codes, but sometimes this PR fails in CI tests, caused by code merge, typical
error is about unknown `env`: `std::env` was delete by this PR.
@alamb If I understood correctly, are you suggesting me update existing
codes that call `env::("*_TEST_DIR")` ?
If so, I'll commit files (including to README.md) for review.
----------------------------------------------------------------
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]
