HyukjinKwon opened a new pull request, #60:
URL: https://github.com/apache/spark-connect-rust/pull/60

   ### What changes were proposed in this pull request?
   
   `resolve_jira_issue` in `dev/merge_connect_rust_pr.py` filtered the SPARK 
project versions to plain `x.y.z` names (`re.match(r"\d+\.\d+\.\d+", ...)`), 
which matches only the core Spark versions (4.2.0, 5.0.0, ...) and skips the 
`connect-rust-*` line. This changes the filter to match 
`connect-rust-\d+\.\d+\.\d+` versions instead.
   
   ### Why are the changes needed?
   
   spark-connect-rust ships on its own `connect-rust-*` version line. With the 
old filter, the merge tool offered and set core Spark fix versions on 
spark-connect-rust JIRAs, which then leaked into the **core Spark** release 
notes. Matching the `connect-rust-*` versions keeps these JIRAs in the 
connect-rust release notes where they belong.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Committer-side merge tooling only.
   
   ### How was this patch tested?
   
   `python3 -m py_compile dev/merge_connect_rust_pr.py`. The regex was verified 
against the current SPARK project versions: it now selects `connect-rust-0.1.0` 
and excludes the core `x.y.z` versions.
   
   This pull request and its description were written by Isaac.
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to