[GitHub] [hadoop] GauthamBanasandra commented on a diff in pull request #5679: HADOOP-18746. Install Python 3 for Windows 10 docker image

["GauthamBanasandra (via GitHub)" <gi...@apache.org>]

GauthamBanasandra commented on code in PR #5679:
URL: https://github.com/apache/hadoop/pull/5679#discussion_r1199758698


##########
dev-support/docker/Dockerfile_windows_10:
##########
@@ -108,6 +108,12 @@ RUN powershell Copy-Item -Path "C:\LibXXHash\usr\bin\*.dll" -Destination "C:\Pro
 RUN powershell Copy-Item -Path "C:\LibZStd\usr\bin\*.dll" -Destination "C:\Program` Files\Git\usr\bin"
 RUN powershell Copy-Item -Path "C:\RSync\usr\bin\*" -Destination "C:\Program` Files\Git\usr\bin"
 
+# Install Python 3.10.11.

Review Comment:
   I normally choose a version that's less recent so that it's mature and has got the appropriate fixes (if any that were discovered upon release).
   1. `3.10.11` -> Would've gone through 11 iterations after release, hopefully containing the bug fixes from the time when `3.10.0` was released.
   2. `3.12.0` -> Is just the first release and could potentially have some bugs.
   
   This is just a practice that I follow. I don't mind bumping up the version to `3.12.0` if you insist.



-- 
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: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org