aIbrahiim commented on PR #39436: URL: https://github.com/apache/beam/pull/39436#issuecomment-5059894432
> This change made some version specific logic which adds maintenance burden in the future. I see currently the workflow is green and work fine > > ======== > > I may have missed something, but currently the workflow already runs each python version in different matrix: > > https://github.com/apache/beam/actions/runs/29992852197 > > as one can see, Python 3.14 and 3.14 distroless have they own job: > > https://github.com/apache/beam/actions/runs/29992852197/job/89159380239 > > https://github.com/apache/beam/actions/runs/29992852197/job/89159380379 > > ======== > > If the concern is on postcommit tests, we can make Python ARM PostCommits to use prebuild containers (gcr.io/apache-beam-testing/beam_python*_sdk:latest) to save time. Thank you for the feedback @Abacn okay right each Python version already has its own matrix job, I may have overstated that part and in this PR I was trying to only change when py314 runs so today everything starts together and shares the same runners, and this would make py314 wait until the main matrix finishes But I got it as this adds version-specific logic and extra maintenance, and if Snapshots is already green, the benefit is pretty small I will also consider reusing the prebuilt images and check in a new PR. -- 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]
