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

RANGER-3409: remove org.codehaus.jackson ver 1.9 / add com.fasterxml.jackson ver 2.17 #312

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

Conversation

nblagodarnyi
Copy link

What changes were proposed in this pull request?

According to https://issues.apache.org/jira/browse/RANGER-3409 all appearances of old org.codehaus.jackson dependencies (ver.1.9.13) were replaced by corresponding new versions from com.fasterxml.jackson.

How was this patch tested?

  • Locally tested with existing unit/intergration tests in project with different Hadoop versions.
export JAVA_HOME=/path/to/java11;
mvn clean package -Dhadoop.version=3.3.6;  # with latest stable release
mvn clean package; # with default Hadoop version
  • In physical test environment where Hadoop 3.3.6 / Hive 4 are installed on Ubutnu 20.

@sercanCyberVision
Copy link

sercanCyberVision commented May 3, 2024

I have build your branch with Docker option and checked the UI, I see below error in multiple endpoints:
image

For example:
http://[host:port]/index.html#/users/usertab
http://[host:port]/index.html#/reports/audit/admin

I have seen jackson related serialization issues before which affect UI, you can see the details here:
https://issues.apache.org/jira/browse/RANGER-4225
#252

Could you please check the UI in your end just to make sure?

@nblagodarnyi
Copy link
Author

Hello Sercan! Thanks for your reply. I didn't manage to reproduce your issue with UI. I used this command to rebuild ranger from my branch and everything works fine in Docker.
export RANGER_REBUILD=1 && export DOCKER_MAVEN_BUILD=0 && ./ranger_in_docker up

image image image

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