nealrichardson commented on a change in pull request #9610:
URL: https://github.com/apache/arrow/pull/9610#discussion_r588825318
##########
File path: dev/tasks/r/azure.linux.yml
##########
@@ -50,7 +50,12 @@ jobs:
# we have to export this (right?) because we need it in the build env
export ARROW_R_DEV={{ not_cran }}
# Note that ci/scripts/r_test.sh sets NOT_CRAN=true if ARROW_R_DEV=TRUE
- docker-compose run r
+ docker-compose run \
+ -e ARROW_DATASET={{ arrow_dataset|default("ON") }} \
+ -e ARROW_PARQUET={{ arrow_parquet|default("ON") }} \
+ -e ARROW_S3={{ arrow_s3|default("ON") }} \
Review comment:
This should work too right? That way by default we're testing the
default, unset versions.
```suggestion
-e ARROW_DATASET={{ arrow_dataset|default("") }} \
-e ARROW_PARQUET={{ arrow_parquet|default("") }} \
-e ARROW_S3={{ arrow_s3|default("") }} \
```
I'll try it and revert if it's no good.
----------------------------------------------------------------
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
