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

[VL][CI] Fix thrift download failure #5080

Closed
wants to merge 3 commits into from

Conversation

PHILO-HE
Copy link
Contributor

@PHILO-HE PHILO-HE commented Mar 22, 2024

What changes were proposed in this pull request?

Fix CI failure on Centos-7/8.

1. Use another proxy. 2. Add another download link for thrift.
Pre-download thrift tar package into docker and let arrow find it from local path of docker container.

Copy link

Thanks for opening a pull request!

Could you open an issue for this pull request on Github Issues?

https://github.com/apache/incubator-gluten/issues

Then could you also rename commit message and pull request title in the following format?

[GLUTEN-${ISSUES_ID}][COMPONENT]feat/fix: ${detailed message}

See also:

Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE force-pushed the fix-arrow-thrift-download branch from f386097 to 9848d77 Compare March 22, 2024 07:01
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE changed the title [CI] Fix thrift download link for arrow [VL][CI] Fix thrift download link for arrow Mar 22, 2024
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE force-pushed the fix-arrow-thrift-download branch from b6d13fb to 7bff4ab Compare March 22, 2024 09:47
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE force-pushed the fix-arrow-thrift-download branch from 7bff4ab to 4984d0a Compare March 22, 2024 14:02
Copy link

Run Gluten Clickhouse CI

1 similar comment
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE force-pushed the fix-arrow-thrift-download branch from 19ab560 to 8f379ec Compare March 23, 2024 02:03
Copy link

Run Gluten Clickhouse CI

1 similar comment
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE force-pushed the fix-arrow-thrift-download branch from 053145a to 2c1eaf9 Compare March 24, 2024 13:45
Copy link

Run Gluten Clickhouse CI

@PHILO-HE PHILO-HE changed the title [VL][CI] Fix thrift download link for arrow [VL][CI] Fix thrift download failure Mar 25, 2024
@PHILO-HE
Copy link
Contributor Author

Closing as new CI is landed.

@PHILO-HE PHILO-HE closed this Mar 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant