Copilot commented on code in PR #49929:
URL: https://github.com/apache/arrow/pull/49929#discussion_r3311228582
##########
dev/tasks/r/github.linux.arrow.version.back.compat.yml:
##########
@@ -90,10 +90,8 @@ jobs:
- { old_arrow_version: '10.0.1', r: '4.2' }
- { old_arrow_version: '9.0.0', r: '4.2' }
- { old_arrow_version: '8.0.0', r: '4.2' }
- - { old_arrow_version: '7.0.0', r: '4.1' }
- - { old_arrow_version: '6.0.1', r: '4.1' }
- - { old_arrow_version: '5.0.0', r: '4.1' }
- - { old_arrow_version: '4.0.0', r: '4.1' }
+ - { old_arrow_version: '7.0.0', r: '4.2' }
+ - { old_arrow_version: '6.0.1', r: '4.2' }
Review Comment:
The back-compat matrix no longer includes any Arrow versions < 6.0.0 (e.g.,
5.0.0/4.0.0 were dropped), which makes the `Prepare RSPM` step’s `if [
${old_arrow_version_major} -ge 6 ] ... else ...` branch effectively always take
the `>= 6` path. Either re-add the intended <6 entries (if you still want
coverage there) or simplify the workflow by removing the now-dead `< 6`
handling to avoid confusing future maintenance.
--
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]
