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

Exclude avro dependency #366

Closed
wants to merge 2 commits into from

Conversation

kokosing
Copy link
Contributor

Exclude avro dependency

Avro is not used by Ranger and it ships CVEs to the project.

Avro is not used by Ranger and it ships CVEs to the project.
@kokosing
Copy link
Contributor Author

FYI @mneethiraj @ksobolew

@ksobolew
Copy link
Contributor

I can see some references to Avro in plugin-schema-registry and assembly for kms, but indeed, no direct dependencies. I'm not qualified to tell if this means that Avro is being used for something.

@kokosing
Copy link
Contributor Author

@mneethiraj Any chance to land this before release-ing Ranger 2.5.0?

@mneethiraj
Copy link
Contributor

@kokosing - this update will likely not be to be included in Ranger 2.5.0 release; release-candidate #3 is already out for voting. I suggest getting this merged in master branch for 3.0.0 release; if a maintenance release is necessary, 2.5.1 release can be considered.

@mneethiraj
Copy link
Contributor

Patch merged via RANGER-4884.

@mneethiraj mneethiraj closed this Aug 1, 2024
@kokosing
Copy link
Contributor Author

kokosing commented Aug 1, 2024

Thank you!

@kokosing kokosing deleted the origin/master/017_avro branch August 1, 2024 09:02
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.

3 participants