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

move druid-ranger-security from core to contrib #17624

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

clintropolis
Copy link
Member

Description

The druid-ranger-security extension doesn't seem to be actively maintained, and is very large, so this PR moves it from a 'core' extension to a 'contrib' extension.

This extension has quite a large size, so removing it also shrinks the binary distribution quite a bit.

before:

-rw-r--r--   1 clint  staff   652M Jan 14 12:21 apache-druid-32.0.0-SNAPSHOT-bin.tar.gz

after:

-rw-r--r--   1 clint  staff   571M Jan 14 12:07 apache-druid-32.0.0-SNAPSHOT-bin.tar.gz

If any committer wants to actively maintain this extension, I think it should be modified to use the ranger client https://github.com/apache/ranger/tree/master/intg/src/main/java instead of how it currently works, which I think would shrink the footprint in the binary distribution quite a lot.

Release note

druid-ranger-security has been moved from a 'core' extension to a 'contrib' extension to better reflect how actively it is maintained. This means it is no longer bundled with the binary distribution by default, but is still available to download the same as any other community extension.

@clintropolis
Copy link
Member Author

i think we can ignore the coverage failures because it isn't new

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant