You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "dongjoon-hyun (via GitHub)" <gi...@apache.org> on 2024/09/18 21:54:29 UTC

[PR] [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists [spark-docker]

dongjoon-hyun opened a new pull request, #68:
URL: https://github.com/apache/spark-docker/pull/68

   ### What changes were proposed in this pull request?
   
   This PR aims to fix `RELEASE` file to have the correct information in Docker images if exists.
   
   Apache Spark repository already fixed this.
   - https://github.com/apache/spark/pull/42636
   
   ### Why are the changes needed?
   
   To provide a correct information for Spark 3.4+
   
   ### Does this PR introduce _any_ user-facing change?
   
   No.
   
   ### How was this patch tested?
   
   Pass the CIs.


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


Re: [PR] [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists [spark-docker]

Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #68:
URL: https://github.com/apache/spark-docker/pull/68#issuecomment-2359657875

   For the record, the fixed images are released.
   ```
   $ docker run -it --rm apache/spark:4.0.0-preview1 cat /opt/spark/RELEASE
   Spark 4.0.0-preview1 (git revision 7a7a8bc4bab) built for Hadoop 3.4.0
   Build flags: -B -Pyarn -Pkubernetes -Psparkr -Phadoop-3 -Phive -Phive-thriftserver
   ```


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


Re: [PR] [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists [spark-docker]

Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #68:
URL: https://github.com/apache/spark-docker/pull/68#issuecomment-2359607898

   Thank you, @viirya !


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


Re: [PR] [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists [spark-docker]

Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun commented on PR #68:
URL: https://github.com/apache/spark-docker/pull/68#issuecomment-2359607531

   This is irrelevant to the functionality and K8s integration tests. 
   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


Re: [PR] [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists [spark-docker]

Posted by "dongjoon-hyun (via GitHub)" <gi...@apache.org>.
dongjoon-hyun closed pull request #68: [SPARK-44935] Fix `RELEASE` file to have the correct information in Docker images if exists
URL: https://github.com/apache/spark-docker/pull/68


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