RANGER-4076: Support Java 17 for build and runtime#872
RANGER-4076: Support Java 17 for build and runtime#872asf-gitbox-commits merged 15 commits intomasterfrom
Conversation
9755de9 to
68df023
Compare
8d5a3ce to
58be178
Compare
58be178 to
fc89780
Compare
|
Thanks @pradeepagrawal8184 for this patch, I think it is best to update the CI/CD build in this PR by removing builds for JDK8 and JDK11 and adding support for JDK17. The jobs |
fc89780 to
4732fb2
Compare
4732fb2 to
9467111
Compare
|
Current ranger master branch shipped jars vs jar shipped with this branch changes. |
e7cd2ea to
9f9247c
Compare
|
Hi @pradeepagrawal8184, |
@ramackri : Spring upgrade can be handled separately. We can discuss that in the spring upgrade ticket after this PR merge and decide. |
kumaab
left a comment
There was a problem hiding this comment.
I've reviewed the changes partially (since this is a big patch), some JWT related functionalities recently added seems to have been removed, please rebase and take a look. thanks.
890fc0a to
32469c3
Compare
32469c3 to
f3b4e48
Compare
82e52ca to
769b97e
Compare
92adab9 to
8dae3a5
Compare
vyommani
left a comment
There was a problem hiding this comment.
Overall LGTM.
This is a significant and well-executed upgrade to Java 17. Thanks for handling the pom updates, Docker changes, Nashorn removal, Jersey migration, and test fixes across the board.
Minor suggestions:
Consider removing vim from the Docker images if not strictly required.
Ensure the CI workflow now only runs on JDK 17.
Great work @pradeepagrawal8184!
8dae3a5 to
c8c04a8
Compare
c8c04a8 to
21520b7
Compare
What changes were proposed in this pull request?
Support jdk17 build in Ranger. This change shall discontinue build support on jdk8/11
How was this patch tested?
Manual install, start and crud operations on the policy, group, users, service etc.