gaogaotiantian commented on PR #56679: URL: https://github.com/apache/spark/pull/56679#issuecomment-4775385873
Yeah I will delete the unused env vars. The problem for Python version is that the existing `build_and_test.yml` on `branch-3.5` does not support a way to define Python version - https://github.com/apache/spark/blob/03733f171795a9da8fcadf4915d323f442129118/.github/workflows/build_and_test.yml#L441 . For `master`, we use variables like `$PYTHON_TO_TEST` to configure which python to test, and also which docker image should be used. For `branch-3.5`, we use one docker image (based on the branch we passed in) and we hardcode the Python version. If we want to test different python versions on `branch-3.5`, we need to backport some of the CI code in `master` back to `branch-3.5`, which should not be too difficult - at least we only need to do this to `branch-3.5` after the CI decoupling. -- 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]
