[GitHub] [arrow] nealrichardson commented on a change in pull request #9610: ARROW-11735: [R] Allow Parquet and Arrow Dataset to be optional components

[GitBox]

nealrichardson commented on a change in pull request #9610:
URL: https://github.com/apache/arrow/pull/9610#discussion_r587897840



##########
File path: dev/tasks/r/azure.linux.yml
##########
@@ -47,6 +47,9 @@ jobs:
         export R_ORG={{ r_org }}
         export R_IMAGE={{ r_image }}
         export R_TAG={{ r_tag }}
+        export ARROW_DATASET={{ arrow_dataset|default("ON") }}
+        export ARROW_PARQUET={{ arrow_parquet|default("ON") }}

Review comment:
       Unfortunately this didn't pass through all the way: 
https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=1721&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=6c939d89-0d1a-51f2-8b30-091a7a82e98c&l=240
   
   I believe what you instead want is to pass them into docker-compose below, 
something like `docker-compose -e ARROW_DATASET={{ arrow_dataset|default("ON") 
}} -e ARROW_PARQUET={{ arrow_parquet|default("ON") }} -e ARROW_S3={{ 
arrow_s3|default("ON") }} r`




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