-
Notifications
You must be signed in to change notification settings - Fork 695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CI] Fix Connection time out in Python workflow #1651
Conversation
@jbampton @jiayuasu Not sure why the lint is not passing. It is passing on my fork's PR: furqaankhan#2 |
Hey @furqaankhan the Basically I have the fix in #1654 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This value /home/runner/.local/share/virtualenvs/python-${PYTHON_VERSION}/lib/python${PYTHON_VERSION}/site-packages/pyspark
is so long. Can you make it a global env?
A quick example about global env from ChatGPT
|
Co-authored-by: Jia Yu <jiayu@wherobots.com>
- spark: '3.0.3' | ||
scala: '2.12.8' | ||
python: '3.7' | ||
hadoop: '2.7' | ||
env: | ||
VENV_PATH: /home/runner/.local/share/virtualenvs/python-${{ matrix.python }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't add the whole path because it would be easier this way to use the existing virtual environment at lines 143 and 158.
Great work! |
Did you read the Contributor Guide?
Is this PR related to a JIRA ticket?
[CI] my subject
What changes were proposed in this PR?
How was this patch tested?
Did this PR include necessary documentation updates?