[beam] branch users/damccorm/addBackOrdering updated: Define var inside loop to avoid async race condition

[damccorm]

This is an automated email from the ASF dual-hosted git repository.

damccorm pushed a commit to branch users/damccorm/addBackOrdering
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/users/damccorm/addBackOrdering 
by this push:
     new 373413010fd Define var inside loop to avoid async race condition
373413010fd is described below

commit 373413010fdaab8019cba0b1df92bab8beb5a5a2
Author: Danny McCormick <dannymccorm...@google.com>
AuthorDate: Thu Aug 17 11:31:09 2023 -0400

    Define var inside loop to avoid async race condition
---
 sdks/python/container/build.gradle | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sdks/python/container/build.gradle 
b/sdks/python/container/build.gradle
index c11952c7d47..06b1ea918c7 100644
--- a/sdks/python/container/build.gradle
+++ b/sdks/python/container/build.gradle
@@ -45,10 +45,11 @@ tasks.register("buildAll") {
 }
 
 for(int i=min_python_version; i<=max_python_version; ++i) {
+  String min_version = "3" + min_python_version
   String cur = "3" + i
   String prev = "3" + (i-1)
   tasks.register("push" + cur) {
-    if (i != min_python_version) {
+    if (cur != min_version) {
       // Enforce ordering to allow the prune step to happen between runs.
       // This will ensure we don't use up too much space (especially in CI 
environments)
       mustRunAfter(":sdks:python:container:push" + prev)