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.

Sign up for GitHub

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

Jump to bottom

Upgrade Alluxio to 312 #22452

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Upgrade Alluxio to 312 #22452

merged 1 commit into from
Apr 9, 2024

Conversation

zacw7
Copy link
Member

@zacw7 zacw7 commented Apr 9, 2024

This upgraded version enhances cache TTL: don't load cache pages which exceed the TTL from local file after the worker restart.

== RELEASE NOTES ==

General Changes
* Upgrade Alluxio to 312.

@zacw7 zacw7 marked this pull request as ready for review April 9, 2024 00:48
@zacw7 zacw7 requested a review from a team as a code owner April 9, 2024 00:48
@zacw7 zacw7 requested a review from presto-oss April 9, 2024 00:48
@zacw7 zacw7 merged commit c9d1a8c into prestodb:master Apr 9, 2024
57 checks passed
@zacw7 zacw7 deleted the alluxio-312 branch April 9, 2024 02:04
@gggrace14
Copy link
Contributor

@zacw7 Can you help test it with prod traffic before this change hits the prod?

@wanglinsong wanglinsong mentioned this pull request May 1, 2024
Merged
48 tasks
@zacw7 zacw7 requested a review from beinan June 7, 2024 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@NikhilCollooru NikhilCollooru NikhilCollooru approved these changes

@gggrace14 gggrace14 Awaiting requested review from gggrace14

@amitkdutta amitkdutta Awaiting requested review from amitkdutta

@presto-oss presto-oss Awaiting requested review from presto-oss presto-oss is a code owner automatically assigned from prestodb/committers

@beinan beinan Awaiting requested review from beinan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@zacw7 @gggrace14 @NikhilCollooru