Update pkg/common/flyte_url.go

Co-authored-by: Haytham Abuelfutuh <[email protected]> Signed-off-by: Yee Hing Tong <[email protected]>
flyteorg · May 5, 2023 · 19b7443 · 19b7443
1 parent 5533ae8
commit 19b7443
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/pkg/common/flyte_url.go b/pkg/common/flyte_url.go
@@ -47,10 +47,10 @@ func ParseFlyteURL(flyteURL string) (core.NodeExecutionIdentifier, *int, Artifac
 	executionID := matches["exec"]
 	nodeID := matches["node"]
 	var attempt *int // nil means node execution, not a task execution
-	if matches["attempt"] != "" {
-		a, err := strconv.Atoi(matches["attempt"])
+	if attempt := matches["attempt"]; len(attempt) > 0 {
+		a, err := strconv.Atoi(attempt)
 		if err != nil {
-			return core.NodeExecutionIdentifier{}, nil, ArtifactTypeUndefined, fmt.Errorf("failed to parse attempt, %s", err)
+			return core.NodeExecutionIdentifier{}, nil, ArtifactTypeUndefined, fmt.Errorf("failed to parse attempt [%v], %v", attempt, err)
 		}
 		attempt = &a
 	}