-
Notifications
You must be signed in to change notification settings - Fork 458
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
Conversation
@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. |
@wForget, CI has passed. Please revert the test code. Then, I will merge the pr. Thanks! |
Reverted the test code, |
* [MINOR] Remove duplicate mvn packages (#6560) * [VL] Improve package scripts (#6569) * [GLUTEN-6569][VL] Delete unnecessary gcc9 enabling from package script (#6730) * [GLUTEN-7118][VL] Fix duckdb target issue when vcpkg is enabled (#7117) --------- Co-authored-by: Zhen Wang <643348094@qq.com> Co-authored-by: PHILO-HE <feilong.he@intel.com>
What changes were proposed in this pull request?
Address #6569 (comment)
How was this patch tested?
Velox backend weekly job
GA