gaogaotiantian commented on PR #56517: URL: https://github.com/apache/spark/pull/56517#issuecomment-4712687082
Hi @sarutak , if it's possible, could you merge this to the older branches as well. I don't think we are supposed to use version pins like `v3` for apache projects. It was okay because the yml file is never used for older branches. Once the branch is cut, the `build_and_test.yml` file would only be used when a new commit is pushed to that branch. However, we are moving to a point where `build_and_test.yml` on that branch is the only CI source for that branch, so the scheduled CI is using the `build_and_test.yml` file from that branch, instead of master. This change will effectively make `build_and_test.yml` on every branch active. We will need to update those files in the future so maybe it's nice to just update them now. Even if we don't think about scheduled CI, the current commit to the branch will trigger CI based on `build_and_test.yml` on the old branch so maybe it's good to just update them? -- 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]
