Abacn commented on code in PR #26913:
URL: https://github.com/apache/beam/pull/26913#discussion_r1207445252
##########
sdks/python/container/piputil.go:
##########
@@ -148,8 +148,11 @@ func findBeamSdkWhl(files []string, acceptableWhlSpecs
[]string) string {
// SDK from source tarball provided in sdkSrcFile.
func installSdk(files []string, workDir string, sdkSrcFile string,
acceptableWhlSpecs []string, required bool) error {
sdkWhlFile := findBeamSdkWhl(files, acceptableWhlSpecs)
+
if sdkWhlFile != "" {
- err := pipInstallPackage(files, workDir, sdkWhlFile, false,
false, []string{"gcp"})
+ // by default, pip rejects to install wheel if same version
already installed
+ isDev := strings.Contains(sdkWhlFile, ".dev")
Review Comment:
I see, I might have checked streaming pipeline (which is the case for #26908
that discovered
checked a batch python postcommit job it indeed run on sibling worker
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
