[SPARK-51493] Refine merge_spark_pr.py to use connect-swift-x.y.z version

dongjoon-hyun · dongjoon-hyun · commit 593c81078a28 · 2025-03-12T14:17:03.000-07:00
### What changes were proposed in this pull request? This PR aims to refine `merge_spark_pr.py` to use `connect-swift-x.y.z` version pattern. ### Why are the changes needed? To avoid using the version of Apache Spark repository. ### Does this PR introduce _any_ user-facing change? No, this is an dev script. ### How was this patch tested? Manual review. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #14 from dongjoon-hyun/SPARK-51493. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
diff --git a/dev/merge_spark_pr.py b/dev/merge_spark_pr.py
@@ -311,7 +311,9 @@ def resolve_jira_issue(merge_branches, comment, default_jira_id=""):
     versions = [
         x
         for x in versions
-        if not x.raw["released"] and not x.raw["archived"] and re.match(r"\d+\.\d+\.\d+", x.name)
+        if not x.raw["released"]
+        and not x.raw["archived"]
+        and re.match(r"connect-swift-\d+\.\d+\.\d+", x.name)
     ]
     versions = sorted(versions, key=lambda x: x.name, reverse=True)
 

Original file line number	Diff line number	Diff line change
`@@ -311,7 +311,9 @@ def resolve_jira_issue(merge_branches, comment, default_jira_id=""):`
`311`	`311`	`versions = [`
`312`	`312`	`x`
`313`	`313`	`for x in versions`
`314`		`- if not x.raw["released"] and not x.raw["archived"] and re.match(r"\d+\.\d+\.\d+", x.name)`
	`314`	`+ if not x.raw["released"]`
	`315`	`+ and not x.raw["archived"]`
	`316`	`+ and re.match(r"connect-swift-\d+\.\d+\.\d+", x.name)`
`315`	`317`	`]`
`316`	`318`	`versions = sorted(versions, key=lambda x: x.name, reverse=True)`
`317`	`319`