AydarZaynutdinov commented on a change in pull request #16277:
URL: https://github.com/apache/beam/pull/16277#discussion_r772446962
##########
File path: playground/backend/cmd/server/http.go
##########
@@ -33,3 +49,28 @@ func listenHttp(ctx context.Context, errChan chan error,
envs environment.Networ
return
}
}
+
+// isReady checks the number of already working code processing.
+// It counts by the number of the
/path/to/workingDir/executable_files/{pipelineId} folders.
+// If it is equals or more than numOfParallelJobs, then returns false.
+// If it is less than numOfParallelJobs, then returns true.
+func isReady(workingDir string, numOfParallelJobs int) bool {
+ // TODO add getting of dir executable_files from environments.
+ baseFileFolder := filepath.Join(workingDir, "executable_files")
+ _, err := os.Stat(baseFileFolder)
+ if os.IsNotExist(err) {
+ return true
+ }
+
+ dirEntries, err := os.ReadDir(baseFileFolder)
+ if err != nil {
+ logger.Errorf("Readiness: Error during read %s: %s",
baseFileFolder, err.Error())
+ return false
+ }
+
+ if len(dirEntries) >= numOfParallelJobs {
Review comment:
Added check for the `numOfParallelJobs`'s value. Now if it <= 0 then the
server uses the default value.
--
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]
