-
Notifications
You must be signed in to change notification settings - Fork 956
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
Upgrade the setup-go and checkout versions in the action && fix Spark CI #2938
Upgrade the setup-go and checkout versions in the action && fix Spark CI #2938
Conversation
Signed-off-by: wangyang <wangyang8126@gmail.com>
a3ef989
to
8eaf054
Compare
Signed-off-by: Yikun Jiang <yikunkero@gmail.com>
/lgtm |
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.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: william-wang, Yikun The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Run CI code verify, and often encounter errors when installing go through action/setup-go@v2. I don’t know the reason for now.
I checked set-go/tags and the latest version is v4. You can try to upgrade the latest version to see if it can solve the current problem.
The current pr repairs the CI of
Code Verify
, butE2E Spark Integration Test
CI will still report an error;#2934 repaired
E2E Spark Integration Test
CI, butCode Verify
CI will report an errorCherry-pick the submission of #2934 to the current pr, fix the CI problem together, and avoid the deadlock between the two prs.