You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/02/20 16:46:36 UTC

[GitHub] [spark] khalidmammadov opened a new pull request #35583: [SPARK-38261][INFRA] Add missing R packages from base image

khalidmammadov opened a new pull request #35583:
URL: https://github.com/apache/spark/pull/35583


   Current GitHub workflow job **Linters, licenses, dependencies and documentation generation** is missing R packages to complete Documentation and API build.
   
   **Build and test** -  is not failing as these packages are installed on the base image.
   
   We need to keep them in-sync IMO with the base image for easy switch back to ubuntu runner when ready.
   
   Reference: [**The base image**](https://hub.docker.com/layers/dongjoon/apache-spark-github-action-image/20220207/images/sha256-af09d172ff8e2cbd71df9a1bc5384a47578c4a4cc293786c539333cafaf4a7ce?context=explore)
   
   
   ### What changes were proposed in this pull request?
   Adding missing packages to the workflow file 
   
   
   ### Why are the changes needed?
   To make them inline with the base image config and make the job task **complete** for standalone execution (i.e. without this image)
   
   
   ### Does this PR introduce _any_ user-facing change?
   No
   
   
   ### How was this patch tested?
   GitHub builds and in the local Docker containers
   


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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

[GitHub] [spark] khalidmammadov commented on pull request #35583: [SPARK-38261][INFRA] Add missing R packages from base image

Posted by GitBox <gi...@apache.org>.
khalidmammadov commented on pull request #35583:
URL: https://github.com/apache/spark/pull/35583#issuecomment-1046276808


   Related: https://github.com/apache/spark/pull/35516


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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

[GitHub] [spark] AmplabJenkins commented on pull request #35583: [SPARK-38261][INFRA] Add missing R packages from base image

Posted by GitBox <gi...@apache.org>.
AmplabJenkins commented on pull request #35583:
URL: https://github.com/apache/spark/pull/35583#issuecomment-1046383986


   Can one of the admins verify this patch?


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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

[GitHub] [spark] HyukjinKwon commented on pull request #35583: [SPARK-38261][INFRA] Add missing R packages from base image

Posted by GitBox <gi...@apache.org>.
HyukjinKwon commented on pull request #35583:
URL: https://github.com/apache/spark/pull/35583#issuecomment-1046399766


   Merged to master.


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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

[GitHub] [spark] HyukjinKwon closed pull request #35583: [SPARK-38261][INFRA] Add missing R packages from base image

Posted by GitBox <gi...@apache.org>.
HyukjinKwon closed pull request #35583:
URL: https://github.com/apache/spark/pull/35583


   


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

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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