Remove Rubix caching #20102
Merged
Remove Rubix caching #20102
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mosabua
previously requested changes
Dec 13, 2023
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
d645430 to
7719227
Compare
|
@mosabua Rubix and Alluxio are unpatched and unmaintained dependencies that have high score CVEs. Rubix doesn't work so keeping it doesn't bring any value other than false impression that hive caching is actually supported. Yeah, I'd love to see new Alluxio land as it has a higher chance of being maintained, supported and actually working. |
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
7719227 to
4c6dce2
Compare
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
b4b33bd to
ab9db53
Compare
|
Thank you! |
raunaqmorarka
approved these changes
Jan 2, 2024
|
Imho we should merge - provided the CI failures are false alarms. |
|
We’re going to merge it after the Alluxio changes get in. |
|
Sounds good @martint - once we have coverage with Alluxio caching for Delta Lake from the current PR or once we have it also for Hive? |
Merged
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
ab9db53 to
3635e6a
Compare
|
Rebased. |
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
3635e6a to
711690b
Compare
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
711690b to
9214876
Compare
|
Rebased |
Rubix was abandoned 3 years ago, it's broken in majority of production cases with frequent cluster crashes, it has number of high impact CVEs and we are planning to replace it with Alluxio caching. Keeping Rubix in its current form brings more harm than good.
wendigo
force-pushed
the
serafin/drop-rubix
branch
from
9214876 to
b1a4e5f
Compare
raunaqmorarka
approved these changes
Feb 13, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rubix was abandoned 3 years ago, it's broken in majority of production cases with frequent cluster crashes, it has number of high impact CVEs and we are planning to replace it with Alluxio caching. Keeping Rubix in its current form brings more harm than good.
Description
Additional context and related issues
Release notes
( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
(x) Release notes are required, with the following suggested text: