[GitHub] [spark] ifilonenko commented on pull request #28594: [SPARK-31780][K8S][TESTS] Add R test tag to exclude R K8s image building and test

[GitBox <gi...@apache.org>]

ifilonenko commented on pull request #28594:
URL: https://github.com/apache/spark/pull/28594#issuecomment-662613310


   Is there a reason we are skipping this test? We are doing: 
   ```
   ./dev/make-distribution.sh --name ${DATE}-${REVISION} --r --pip --tgz -DzincPort=${ZINC_PORT} \
        -Phadoop-2.7 -Pkubernetes -Pkinesis-asl -Phive -Phive-thriftserver
   retcode=$?
   ```
   so we are already building the distribution with R support. What do we gain from skipping this test? I would have preferred to keep this test tbh, (so as to have greater coverage). 


----------------------------------------------------------------
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:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org