Skip to content
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

[GLUTEN-6569][FOLLOWUP][VL] Delete unnecessary gcc9 enable of package script #6730

Merged
merged 3 commits into from
Aug 8, 2024

Conversation

wForget
Copy link
Member

@wForget wForget commented Aug 6, 2024

What changes were proposed in this pull request?

Address #6569 (comment)

How was this patch tested?

Velox backend weekly job GA

@github-actions github-actions bot added the BUILD label Aug 6, 2024
Copy link

github-actions bot commented Aug 6, 2024

#6569

@PHILO-HE
Copy link
Contributor

PHILO-HE commented Aug 6, 2024

@wForget, could you add some code (e.g., echo something) in velox_weekly.yml? It's just for triggering the job for verification and will be reverted before merging. Thanks!

@github-actions github-actions bot added the INFRA label Aug 6, 2024
@wForget
Copy link
Member Author

wForget commented Aug 6, 2024

@wForget, could you add some code (e.g., echo something) in velox_weekly.yml? It's just for triggering the job for verification and will be reverted before merging. Thanks!

Good way, I will try to check this.

@PHILO-HE
Copy link
Contributor

PHILO-HE commented Aug 7, 2024

@wForget, CI has passed. Please revert the test code. Then, I will merge the pr. Thanks!

@github-actions github-actions bot removed the INFRA label Aug 8, 2024
@wForget
Copy link
Member Author

wForget commented Aug 8, 2024

@wForget, CI has passed. Please revert the test code. Then, I will merge the pr. Thanks!

Reverted the test code,

@PHILO-HE PHILO-HE merged commit 5cce283 into apache:main Aug 8, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants