kiszk commented on a change in pull request #7507: URL: https://github.com/apache/arrow/pull/7507#discussion_r576988294
########## File path: ci/scripts/integration_arrow.sh ########## @@ -24,9 +24,16 @@ source_dir=${1}/cpp build_dir=${2}/cpp gold_dir_0_14_1=$arrow_dir/testing/data/arrow-ipc-stream/integration/0.14.1 gold_dir_0_17_1=$arrow_dir/testing/data/arrow-ipc-stream/integration/0.17.1 +gold_dir_1_0_0_le=$arrow_dir/testing/data/arrow-ipc-stream/integration/1.0.0-littleendian +gold_dir_1_0_0_be=$arrow_dir/testing/data/arrow-ipc-stream/integration/1.0.0-bigendian pip install -e $arrow_dir/dev/archery archery integration --with-all --run-flight \ --gold-dirs=$gold_dir_0_14_1 \ --gold-dirs=$gold_dir_0_17_1 \ + +# TODO: support other languages +archery integration --with-cpp=1 --run-flight \ Review comment: Oh, sorry, I should mention you [here](https://github.com/apache/arrow/pull/7507#issuecomment-774403485) The answer is yes. It fails on Java at least. Sure, I will update `runner.py` instead of `integration_arrow.sh`. ---------------------------------------------------------------- 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: us...@infra.apache.org